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

fix(t8s-cluster/clusterClass): fix redundant rolling of machines #443

Merged
merged 1 commit into from
Jul 24, 2023

Conversation

cwrau
Copy link
Member

@cwrau cwrau commented Jun 13, 2023

Changing the openStackMachineTemplates resulted in a rolling of the machines in the old version and then again in the new version.

As now the openStackMachineTemplates are not touched this doesn't happen anymore.

@cwrau cwrau requested a review from marvinWolff as a code owner June 13, 2023 13:50
@cwrau cwrau requested a review from a user June 13, 2023 13:50
@cwrau cwrau enabled auto-merge (squash) June 13, 2023 13:50
@teutonet-bot
Copy link
Contributor

🤖 I have diffed this beep boop

"/$namespace/$kind/$name.yaml" for normal resources
"/$namespace/HelmRelease/$name/$namespace/$kind/$name.yaml" for HelmReleases <- this is recursive
'null' means it's either cluster-scoped or it's in the default namespace for the HelmRelease

charts/t8s-cluster/ci/artifacthub-values.yaml

charts/t8s-cluster/ci/cidrs-values.yaml

charts/t8s-cluster/ci/injectedCertificateAuthorities-values.yaml

charts/t8s-cluster/ci/securityGroups-values.yaml

charts/t8s-cluster/ci/calico-values.yaml

charts/t8s-cluster/ci/bastion-values.yaml

charts/t8s-cluster/values.yaml

@ghost
Copy link

ghost commented Jun 14, 2023

Being able to configure the image name template is a very nice bonus!

Changing the openStackMachineTemplates resulted in a rolling of the
machines in the old version and then again in the new version.

As now the openStackMachineTemplates are not touched this doesn't happen
anymore.
@cwrau cwrau force-pushed the fix/t8s-cluster-redundant-rolling-of-machines branch from aa94ceb to b0faf19 Compare June 14, 2023 13:19
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.

3 participants