-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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: broadcast when pod is pushed back to activeQ directly in AddUnschedulableIfNotPresent #119784
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. |
/retest |
The test needs to be in this PR if you intend to add it to the milestone and/or cherry-pick. Is this a regression? |
…hedulableIfNotPresent
Added the test as suggested.
It's a regression by the queueing hint, I believe it's not happening in older versions that don't have the queueing hint change. So, no need to cherry-pick this to older versions, but need to |
/retest |
/kind regression @Huang-Wei can you add to the milestone? |
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
[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 |
/milestone v1.28 |
/lgtm |
LGTM label has been added. Git tree hash: 83e049cf86a566854d74544400cf9b28b2926678
|
/retest |
1 similar comment
/retest |
What type of PR is this?
/kind bug
What this PR does / why we need it:
p.cond.Broadcast()
when pod is pushed back to activeQ directly in AddUnschedulableIfNotPresent.Otherwise, the Pod could stick until another Pod is pushed to activeQ. In other words, if the cluster is super small and only one Pod is unschedulable now, the Pod won't be scheduled forever.
The scenario facing this bug will be covered in test added in #119105
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
@kubernetes/sig-scheduling-leads I'd like to get this in v1.28. 🙏
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: