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 double lock and excercise its codepath in tests #114459
fix double lock and excercise its codepath in tests #114459
Conversation
/sig api-machinery |
/retest |
looks like github is making prow unhappy:
/retest |
/lgtm go ahead and open the cherry-pick to release-1.26 |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alexzielenski, liggitt 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 |
would cause a deadlock if an aggregated apiserver decided to respect ETags for its aggregated discovery document,
09fdda6
to
7510afb
Compare
/retest |
/lgtm |
…f-#114459-upstream-release-1.26 Automated cherry pick of #114459: fix double lock and excercise its codepath in tests
/kind bug
What this PR does / why we need it:
This removes a duplicate lock on the 304 Not Modified codepath likely forgotten to be removed during a refactor.
It is probable that no users have discovered this bug, as it occurs only if the user has an aggregated APIService which makes use of aggregated discovery, which was just released a few days ago.
Also includes a reproducer unit test which fails easily without the change, and is now fixed.
More information in #114442
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: