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
Adding a deprecating featurer gate to fix cost #124675
Conversation
test/integration/apiserver/cel/validatingadmissionpolicy_test.go
Outdated
Show resolved
Hide resolved
This PR may require API review. If so, when the changes are ready, complete the pre-review checklist and request an API review. Status of requested reviews is tracked in the API Review project. |
60f2911
to
81f1682
Compare
LGTM label has been added. Git tree hash: adf0304313b114a88eec7b9c468f1b715d93f8f3
|
/hold in case you want to squash |
Thank you! Squashed :) |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cici37, jpbetz, 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 |
/hold I had one more question... if the gates default on, do unit/integration tests still pass or are there some that need to explicitly assert non-strict cost? |
Updates: All CI passed. I am dropping the temp commit and squash the changes. |
/retest |
@liggitt I believe the concern has been addressed and tested in CI. Would you mind taking a look when have time? Thank you! |
/lgtm |
LGTM label has been added. Git tree hash: 09a688fa6e0d56313aae5eaee5c6ef176aec0209
|
What type of PR is this?
/kind bug
/kind regression
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #124542
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: