-
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
Automated cherry pick of #117194: Revert "Optimization on running prePreEnqueuePlugins #117308
Automated cherry pick of #117194: Revert "Optimization on running prePreEnqueuePlugins #117308
Conversation
…ds into activeQ" This reverts commit c01fa82.
This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/sig scheduling |
/cc kubernetes/release-managers |
/approve |
LGTM label has been added. Git tree hash: 99f9515c2a74249fc3f4e536f4cc738b2b1d7e92
|
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.
/approve
CHANGELOG/CHANGELOG-1.27.md
Outdated
In v1.26.0, we've found the bug that the PreEnqueue extension point doesn't work for Pods going to activeQ through backoffQ. | ||
It doesn't affect any of the vanilla Kubernetes behavior, but, may break custom PreEnqueue plugins. | ||
|
||
The cause PR is [reverted](https://github.com/kubernetes/kubernetes/pull/117194) by v1.26.1. |
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 think this should mention v1.27.0 and v1.27.1 if I'm not mistaken.
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.
fixed
36036d1
to
3036cc4
Compare
Change-Id: I4d38ed15c492abbfafda770fcb6aed2b37e0ac24
3036cc4
to
1601bb7
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.
Only changelog changed:
/lgtm
LGTM label has been added. Git tree hash: ebe546879e2254715d25744aa9bc6d69b0e42dba
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alculquicondor, dims, xmudrii 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 |
/hold cancel |
Cherry pick of #117194 on release-1.27.
#117194: Revert "Optimization on running prePreEnqueuePlugins
For details on the cherry pick process, see the cherry pick requests page.