Skip to content

KEP-5883: Resolve remaining wording comments #5912

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.

Sign up for GitHub

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

Merged
k8s-ci-robot merged 2 commits into from
Feb 11, 2026
Merged

KEP-5883: Resolve remaining wording comments #5912

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a39d6d7
Resolve remaining wording comments for KEP-5883
mm4tt Feb 11, 2026
d431c89
Remove SchedulingGate condition which turned out non-trivial to imple…
mm4tt Feb 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions keps/sig-scheduling/5832-decouple-podgroup-api/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -321,8 +321,6 @@ type PodGroupStatus struct {
// - "Scheduled": All required pods have been successfully scheduled.
// - "Unschedulable": The PodGroup cannot be scheduled due to resource constraints,
// affinity/anti-affinity rules, or insufficient capacity for the gang.
// - "SchedulingGated": One or more pods in the PodGroup have scheduling gates
// that must be cleared before scheduling can proceed.
// - "Preempted": The PodGroup was preempted to make room for higher-priority workloads.
// - "Timeout": The PodGroup failed to schedule within the configured timeout.
//
Expand Down Expand Up @@ -385,7 +383,7 @@ For alpha, we will introduce `Conditions` field with the `PodGroupScheduled` con

`PodGroup` status mirrors `Pod` status semantics:
- If pods are unschedulable(i.e., timeout, resources, affinity, etc.), the scheduler updates the `PodGroupScheduled` condition to `False` and sets the reason fields accordingly.
- If pods are scheduled, the scheduler updates the `PodGroupScheduled` condition to `True` after the last pod in the gang completes binding.
- If pods are scheduled, the scheduler updates the `PodGroupScheduled` condition to `True` after the group got accepted by the Permit phase.

For basic scheduling policy, when the pod related to the `PodGroup` gets scheduled, the scheduler updates the `PodGroupScheduled` condition to `True`.

Expand All @@ -401,7 +399,7 @@ The kube-scheduler will add a new informer to watch `PodGroup` objects and stop

#### GangScheduling plugin

The GangScheduling plugin will maintain a lister for `PodGroup` and check if the `PodGroup` object exists along with the `Workload` object. This is in addition to the following changes:
The GangScheduling plugin will maintain a lister for `PodGroup` and check if the `PodGroup` object exists. This is in addition to the following changes:

**1. PreEnqueue**: The extension will check if the `PodGroup` object exist. If not, it will return `UnschedulableAndUnresolvable` status. Then check if the Pod scheduling requirement is met for gang scheduling (based on minCount value in `PodGroup` object).

Expand Down