-
Notifications
You must be signed in to change notification settings - Fork 405
Add exceptions from v1.33 Code freeze #2753
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 exceptions from v1.33 Code freeze #2753
Conversation
codeFreeze: | ||
|
||
- name: "DRA: structured parameters" | ||
issue: 4381 |
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.
Did they just link the wrong KEP in the code PR? https://github.com/kubernetes/enhancements/tree/master/keps/sig-scheduling/4816-dra-prioritized-list
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.
I’m not entirely sure, but I followed the exception request (here) that was raised to gather the details for the PR and KEP number.
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.
Given the justification given in exception request about moving from beta to GA, that would correspond to KEP-4381, while the other KEP-4816 mentioned in the code PR is still in alpha for this release.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dipesh-rawat, npolshakova 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 |
What type of PR is this:
/kind documentation
What this PR does / why we need it:
Documenting the Code freeze exception requests received in the v1.33 release cycle, after code and test freeze was in place.
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
cc: @npolshakova @neoaggelos