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

feat(t8s-cluster/workload-cluster): add a higher timeout for upgrading #567

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

cwrau
Copy link
Member

@cwrau cwrau commented Sep 12, 2023

during cluster upgrades the rolling nodes can lead to upgrade retries exhausted because the daemonsets can't roll quick enough

during cluster upgrades the rolling nodes can lead to `upgrade retries
exhausted` because the daemonsets can't roll quick enough
@tasches tasches force-pushed the feat/t8s-cluster-adjust-upgrade-timeouts branch from 95f7d14 to 504e4c8 Compare September 13, 2023 09:27
Copy link
Collaborator

@tasches tasches left a comment

Choose a reason for hiding this comment

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

LGTM

@cwrau cwrau merged commit c5cb979 into main Sep 13, 2023
8 checks passed
@cwrau cwrau deleted the feat/t8s-cluster-adjust-upgrade-timeouts branch September 13, 2023 09:31
@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

charts/t8s-cluster/ci/gpu-flavor-values.yaml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants