-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
Update trouble shooting to include the issue of etcd upgrade #43903
Conversation
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
|
||
## Upgrade failed due to etcd hash is not changed | ||
|
||
Only applicable for upgrade from v1.28.0 (v1.28.1, v1.28.2 as well) to v1.28.3 or after. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
chendave wants to merge 1 commit into kubernetes:dev-1.29
i don't know if 1.29 is the right branch for this. the bug is in 1.28.1, .2.
let's wait for feedback from k/website maintainers.
/hold
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Show resolved
Hide resolved
/sig cluster-lifecycle |
@neolit123 thanks for the detail review and all those comments! all were fixed up! |
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
/assign @tengqm PTAL for guidance and approval. we have a bug in 3 PATCH versions of 1.28 that we want to add a troubleshooting entry for. should this PR be against @chendave note that changing the base branch from the github UI will not work, probably. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
content/en/docs/setup/production-environment/tools/kubeadm/troubleshooting-kubeadm.md
Outdated
Show resolved
Hide resolved
For the isse which is reported recently: kubernetes/kubeadm#2957 We'd better to provide some tips to workaround this known issue.
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@sftim @tengqm pls tell if this is the right branch to base against, it's currently on top of 1.29-dev branch, #43903 (comment). thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
LGTM label has been added. Git tree hash: 5c2565c35ddf0160a2bcc3555c55f029e41a0e13
|
I'd suggest we put these contents into |
SGTM, but we don't want this info to appear in the 1.29 upgrade page. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/hold cancel
PTAL again for approval, @tengqm @sftim
thanks
this is still an open Q, but we can clean it up later
#43903 (comment)
We're not using dev-1.28 any more as v1.28 is released. This PR should target main. |
@neolit123: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
For the isse which is reported recently: kubernetes/kubeadm#2957 We'd better to provide some tips to workaround this known issue.
related: kubernetes/kubeadm#2957