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
kubeadm: Delete the dummy Deployment properly #42657
Conversation
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED The following people have approved this PR: deads2k, luxas Needs approval from an approver in each of these OWNERS Files:
We suggest the following people: |
Probably you already know this, foreground GC means the deployment will be deleted after the pods, replicasets are deleted. I don't know kubeadm cares about the order. |
Just rebased without changes to retrigger the (hopefully non-flaky) tests |
Automatic merge from submit-queue |
What this PR does / why we need it:
Which issue this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close that issue when PR gets merged): fixes kubernetes/kubeadm#149Special notes for your reviewer:
Earlier, the Pod created by the Deployment wasn't deleted. With this option it is.
As suggested by @deads2k, thank you!
This is a bug fix for v1.6
Release note:
@mikedanese @jbeda @dmmcquay @pires @errordeveloper @deads2k @caesarxuchao