Skip to content

feat(quickstart): drop flux cli requirement, fix links - #186

Merged
maximiliantech merged 2 commits into
mainfrom
feat/update-quickstart-guide
Jul 31, 2026
Merged

feat(quickstart): drop flux cli requirement, fix links#186
maximiliantech merged 2 commits into
mainfrom
feat/update-quickstart-guide

Conversation

@maximiliantech

@maximiliantech maximiliantech commented Jul 31, 2026

Copy link
Copy Markdown
Member

What this PR does / why we need it:
This PR updates the Platform Owner quick start guide because of a new release of ocpctl@v0.3.0.
We can now drop the requirement to have the flux CLI pre-installed, since ocpctl now takes care of the installation in the platform cluster in the initial setup process (contributed in openmcp-project/ocpctl#92)

Additionally, this PR adds more information how a Platform Owner knows how a Flux instance is healthy/progressing by looking at the Flux.status sub-resource.

Which issue(s) this PR fixes:
NONE

Special notes for your reviewer:

Release note:

Quickstart guide: Drop requirement for pre-installed Flux CLI + fix links

Signed-off-by: Maximilian Techritz <maximilian.techritz@sap.com>

@MichaelSp MichaelSp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image

Comment thread docs/operators/01-quickstart.md Outdated
Comment on lines 60 to 69

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While testing this guide multiple times, this error was not showing up anymore with the latest version of ocpctl@v0.3.0. Can the reviewer please confirm?

@MichaelSp MichaelSp Jul 31, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also ran it 5 times without error. Looks like its a goner => confirmed

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MichaelSp confirms that the error is not showing up anymore ✅

Removed from docs via 66451a4

@maximiliantech
maximiliantech requested a review from MichaelSp July 31, 2026 15:14
MichaelSp
MichaelSp previously approved these changes Jul 31, 2026
…ymore

Signed-off-by: Maximilian Techritz <maximilian.techritz@sap.com>
@maximiliantech
maximiliantech merged commit 7b7adc5 into main Jul 31, 2026
7 checks passed
@maximiliantech
maximiliantech deleted the feat/update-quickstart-guide branch July 31, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants