Skip to content
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

Apply Add-Ons again after cluster upgrade #208

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

Nils98Ar
Copy link
Member

@Nils98Ar Nils98Ar commented Mar 7, 2025

What this PR does / why we need it:

Apply cluster addons after cluster upgrade again to re-run templating with new K8s version.

As far as I understand it the templating is only done with the old K8s version otherwise (before K8s upgrade).

TODOs:

  • squash commits

jschoone and others added 12 commits February 27, 2025 08:12
The values for the metrics server have changed and some are already
outdated.
This smaller version should to the same for this case.

Signed-off-by: Jan Schoone <[email protected]>
Even 1.17 is already available it needs some additional configuration
and would fail here.
Cilium 1.16.x is still supported so we are good here.

Signed-off-by: Jan Schoone <[email protected]>
Signed-off-by: Jan Schoone <[email protected]>
Signed-off-by: Jan Schoone <[email protected]>
Signed-off-by: Jan Schoone <[email protected]>
Signed-off-by: Jan Schoone <[email protected]>
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 <[email protected]>
@jschoone jschoone force-pushed the refactor/modernize_scs_cluster_stacks branch 9 times, most recently from f610df5 to 3d34c07 Compare March 7, 2025 22:45
@jschoone
Copy link
Contributor

jschoone commented Mar 8, 2025

Hi @Nils98Ar, can you explain in the commit or description why this is needed and why it should be solved like this?
Because I'd suggest we check whether we directly find a solution in #211 (formerly part of #203)

@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
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