Skip to content

Use Cluster Addon Stages #211

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 13, 2025
Merged

Use Cluster Addon Stages #211

merged 3 commits into from
Mar 13, 2025

Conversation

jschoone
Copy link
Contributor

@jschoone jschoone commented Mar 7, 2025

The Cluster Stack Operator has the Multi Stage Addon feature to install addons in a specified order. This commit adds the configuration to make use of it at least in a very simple way.

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:

Please do not use "Squash and merge" here

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

TODOs:

  • squash commits
  • include documentation
  • add unit tests

@jschoone jschoone force-pushed the refactor/modernize_scs_cluster_stacks branch 2 times, most recently from f866e24 to 6ce933f Compare March 7, 2025 22:21
@jschoone jschoone force-pushed the feat/use_addon_stages branch 2 times, most recently from 400223d to ba96db3 Compare March 7, 2025 22:28
@jschoone jschoone force-pushed the refactor/modernize_scs_cluster_stacks branch 2 times, most recently from 8ab6819 to fbf8085 Compare March 7, 2025 22:36
@jschoone jschoone changed the title feat(cluster-addon): Use Cluster Addon Stages to instruct CSO (#205) Use Cluster Addon Stages to instruct CSO (#205) Mar 7, 2025
@jschoone jschoone force-pushed the refactor/modernize_scs_cluster_stacks branch 2 times, most recently from f610df5 to 3d34c07 Compare March 7, 2025 22:45
@jschoone jschoone force-pushed the feat/use_addon_stages branch from 80b2c3f to 621d3eb Compare March 7, 2025 22:56
@jschoone jschoone force-pushed the refactor/modernize_scs_cluster_stacks branch from 3f9366a to fbf6fe0 Compare March 10, 2025 11:38
Base automatically changed from refactor/modernize_scs_cluster_stacks to main March 10, 2025 11:39
@jschoone jschoone force-pushed the feat/use_addon_stages branch from f6f60b5 to ca7f3da Compare March 12, 2025 12:43
@jschoone jschoone requested a review from chess-knight March 12, 2025 12:45
@jschoone jschoone force-pushed the feat/use_addon_stages branch from ca7f3da to b3e8dfd Compare March 12, 2025 12:58
jschoone and others added 3 commits March 12, 2025 22:04
The Cluster Stack Operator has the `Multi Stage Addon` feature to
install addons in a specified order. This commit adds the configuration
to make use of it at least in a very simple way.

Signed-off-by: Jan Schoone <jan.schoone@uhurutec.com>
The workload_loadbalancer should configure the lb-provider used in the
Workload Cluster by overwriting the values for OCCM using the addons
controller of the Cluster Stack Operator.
Since we create the cloud-config for the OCCM using the
openstack-csp-helper we need to disable the creation of the
cluster-config secret using the OCCM Helm chart. With that no change of
the cloud-config using the OCCM values will be applies, so the
workload_loadbalancer variable has no effect.

Signed-off-by: Jan Schoone <jan.schoone@uhurutec.com>
@jschoone jschoone force-pushed the feat/use_addon_stages branch from 6821a04 to f4c4547 Compare March 12, 2025 21:05
@jschoone jschoone changed the title Use Cluster Addon Stages to instruct CSO (#205) Use Cluster Addon Stages Mar 13, 2025
@jschoone jschoone merged commit 9da247a into main Mar 13, 2025
4 checks passed
@jschoone jschoone deleted the feat/use_addon_stages branch March 13, 2025 14:20
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.

4 participants