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
Add image removal lock to image server #7850
base: main
Are you sure you want to change the base?
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: saschagrunert 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 |
9fab9fb
to
1254e7b
Compare
3bd60e4
to
4c520c2
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7850 +/- ##
==========================================
- Coverage 49.73% 49.71% -0.03%
==========================================
Files 153 153
Lines 16811 16841 +30
==========================================
+ Hits 8361 8372 +11
- Misses 7408 7425 +17
- Partials 1042 1044 +2 |
5e88f33
to
bd30ba3
Compare
70b6b21
to
8cb5dbe
Compare
A friendly reminder that this PR had no activity for 30 days. |
We now avoid removing images when a sandbox or container creation is in progress. This should close the gap in the outlined corner case in: kubernetes/kubernetes#123631 Refers to kubernetes/kubernetes#123711 as well. Signed-off-by: Sascha Grunert <sgrunert@redhat.com>
8cb5dbe
to
64a606c
Compare
What type of PR is this?
/kind bug
What this PR does / why we need it:
We now avoid removing images when a sandbox or container creation is in progress. This should close the gap in the outlined corner case in:
kubernetes/kubernetes#123631
Refers to kubernetes/kubernetes#123711 as well.
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?