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
Automated cherry pick of #79326: fix: Use correct function to remove etcd member #79331
Automated cherry pick of #79326: fix: Use correct function to remove etcd member #79331
Conversation
Signed-off-by: Brad Beam <brad.beam@talos-systems.com>
/assign @Bubblemelon @rosti @luxas |
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 for the cherry-pick @draveness !
/approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: draveness, rosti The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
flaky test /retest |
@kubernetes/patch-release-team |
/milestone v1.15 @kubernetes/sig-release |
I met the same problem, so I pinged the patch release team which seems the right team to respond. |
Ping the patch-release-team team would work. |
got it now -> |
I am not used to kubeadm internals but I am having an issue with preflight-check error: Does this merged change takes this preflight-check away? Can I safely skip these checks? |
What type of PR is this?
/kind bug
/priority critical-urgent
What this PR does / why we need it:
This uses the correct function name for removing the etcd member from the cluster during a kubeadm reset.
Which issue(s) this PR fixes:
ref: #79326
Special notes for your reviewer:
Does this PR introduce a user-facing change?: