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: register the plugin rejects Pods in WaitOnPermit to UnschedulablePlugins #119785
fix: register the plugin rejects Pods in WaitOnPermit to UnschedulablePlugins #119785
Conversation
Please note that we're already in Test Freeze for the Fast forwards are scheduled to happen every 6 hours, whereas the most recent run was: Sun Aug 6 04:30:20 UTC 2023. |
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. |
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
@@ -731,7 +731,7 @@ func (p *PriorityQueue) AddUnschedulableIfNotPresent(logger klog.Logger, pInfo * | |||
|
|||
// In this case, we try to requeue this Pod to activeQ/backoffQ. | |||
queue := p.requeuePodViaQueueingHint(logger, pInfo, schedulingHint, ScheduleAttemptFailure) | |||
logger.V(6).Info("Pod moved to an internal scheduling queue", "pod", klog.KObj(pod), "event", ScheduleAttemptFailure, "queue", queue, "schedulingCycle", podSchedulingCycle) | |||
logger.V(2).Info("Pod moved to an internal scheduling queue", "pod", klog.KObj(pod), "event", ScheduleAttemptFailure, "queue", queue, "schedulingCycle", podSchedulingCycle, "hint", schedulingHint) |
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.
2 seems excessive. Maybe 3?
LGTM label has been added. Git tree hash: 4b02bc8615f51593fe409df731c88095eec16930
|
This requires a release note |
8325ea6
to
f88f56e
Compare
@alculquicondor added the release note + change the log level as suggested 🙏 |
I tried to adjust it to be in end-user terms. /release-note-edit
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alculquicondor, sanposhiho 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 |
f88f56e
to
cf3f0bd
Compare
rebased to resolve conflict. |
/lgtm |
LGTM label has been added. Git tree hash: 9d9f15e5a89a1471d16f61c5afe4d2a35f47a834
|
What type of PR is this?
/kind feature
What this PR does / why we need it:
A failed Permit plugin gets added to pInfo.UnschedulablePlugins if failed in the Permit extension.
But, it doesn't if failed WaitOnPermit. To make it consistent, this PR changes the scheduler to register the plugin rejects Pods in WaitOnPermit to UnschedulablePlugins.
Which issue(s) this PR fixes:
Ref: #119105 (comment)
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: