-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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 that ephemeral containers were repeatedly created after being manually deleted #124177
base: master
Are you sure you want to change the base?
Conversation
Please note that we're already in Test Freeze for the Fast forwards are scheduled to happen every 6 hours, whereas the most recent run was: Wed Apr 3 02:18:49 UTC 2024. |
Hi @chengjoey. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
/release-note-none |
/ok-to-test |
/test pull-kubernetes-e2e-gce |
Generally, bug fixes should be accompanied by e2e tests or unit tests. You have a pretty clear reproduction tests. Could you explore creating a e2e test for this? https://github.com/kubernetes/kubernetes/blob/master/test/e2e/common/node/ephemeral_containers.go |
/triage accepted |
/cc |
eafbfae
to
ff82264
Compare
/hold |
/kind bug |
ff82264
to
f0518fd
Compare
…stent status after being manually deleted. Signed-off-by: joey <zchengjoey@gmail.com>
f0518fd
to
ba75fb2
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: chengjoey 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 |
/hold cancel |
I added a unit test for this fix, and I tried to write an e2e test, but I found that I didn't find a way to remotely call the cri runtime service. so only unit tests... |
How is this related to #124271? |
I think these should be two different type issues. The current PR is a bug, which is caused by deleting the container at the operation and maintenance level. 124271 is a feature request to allow users to correctly delete ephemeral containers at the API level, deletion was previously prohibited in the cli |
What type of PR is this?
What this PR does / why we need it:
fix that ephemeral containers were repeatedly created and had inconsistent status after being manually deleted.
Which issue(s) this PR fixes:
Fixes #124176
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: