-
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
Modify e2e tests to use priorityClass beta client version & switch priorityClass to beta #63724
Modify e2e tests to use priorityClass beta client version & switch priorityClass to beta #63724
Conversation
b920f01
to
441b6d4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Thanks, @ravisantoshgudimetla!
Thanks for review @bsalamat. Not related to this PR, shouldn't sig-scheduling-approvers be part of test/owners as we have scheduling directory in test/e2e and test/integration? |
Apparently, the convention is to add individuals to the test/OWNERS (with the exception of sig-test-approvers). I can try adding myself there. |
Waiting on this PR to be merged to get sig-cli lgtm- #63928. This PR removes versioned printer.. |
@kubernetes/sig-testing-pr-reviews @kubernetes/sig-cluster-lifecycle-api-reviews for approval |
enabling priority by default is holding until the quota PR is reviewed and merged, correct? |
The quota PR is not a pre-requisite for enabling priority. At least we never had any such plan, but it would be great if both can be done in 1.11. |
/retest |
We need to merge this ASAP to enable Beta version of priority and preemption by default and start running e2e tests with the feature. |
/retest |
1 similar comment
/retest |
/lgtm
agreed. can you link to the PR that enables this feature flag, and in which job? |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bsalamat, liggitt, ravisantoshgudimetla 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 |
[MILESTONENOTIFIER] Milestone Pull Request: Up-to-date for process @bsalamat @deads2k @liggitt @ravisantoshgudimetla Pull Request Labels
|
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions here. |
What this PR does / why we need it:
/cc @bsalamat @aveshagarwal @liggitt
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Part of #57471
Special notes for your reviewer:
Once #57963 merges, this could go in.
Release note: