Skip to content

KEP-4671: Update scheduler docs for pod groups#54666

Merged
k8s-ci-robot merged 1 commit intokubernetes:dev-1.36from
macsko:add_docs_for_workload_cycle
Apr 7, 2026
Merged

KEP-4671: Update scheduler docs for pod groups#54666
k8s-ci-robot merged 1 commit intokubernetes:dev-1.36from
macsko:add_docs_for_workload_cycle

Conversation

@macsko
Copy link
Copy Markdown
Member

@macsko macsko commented Feb 25, 2026

Description

Update scheduler docs for pod groups. Changes to API will be made by #54490

Issue

KEP: kubernetes/enhancements#4671

@k8s-ci-robot k8s-ci-robot added this to the 1.36 milestone Feb 25, 2026
@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language labels Feb 25, 2026
@k8s-ci-robot k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Feb 25, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 25, 2026

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 4e741f8
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/69cf990b881d2a0008bd15cb
😎 Deploy Preview https://deploy-preview-54666--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@macsko
Copy link
Copy Markdown
Member Author

macsko commented Feb 25, 2026

/sig scheduling
/area workload-aware

@k8s-ci-robot k8s-ci-robot added sig/scheduling Categorizes an issue or PR as relevant to SIG Scheduling. area/workload-aware Categorizes an issue or PR as relevant to Workload-aware and Topology-aware scheduling subprojects. labels Feb 25, 2026
@github-project-automation github-project-automation Bot moved this to Needs Triage in SIG Scheduling Feb 25, 2026
@macsko macsko force-pushed the add_docs_for_workload_cycle branch from aa94c71 to 8466f09 Compare April 1, 2026 11:06
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Apr 1, 2026
@macsko macsko marked this pull request as ready for review April 1, 2026 11:08
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 1, 2026
@macsko macsko force-pushed the add_docs_for_workload_cycle branch from 8466f09 to 3bbb44b Compare April 1, 2026 11:16
@macsko
Copy link
Copy Markdown
Member Author

macsko commented Apr 1, 2026

/cc @dom4ha @sanposhiho

1. Finds a feasible node using the standard per-Pod filtering and scoring phases.

* If the Pod fits, it is temporarily assumed and reserved on the selected node until the end of the scheduling algorithm.
* If the Pod cannot fit, the scheduler attempts preemption by running the default `PostFilter` extension point.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do we mean by "default PostFilter extension point" here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good question, removed the "default"

rather than a single-cycle approach.
It utilizes the [PodGroup scheduling](/docs/concepts/scheduling-eviction/podgroup-scheduling/) cycle to make a single,
atomic scheduling decision. The `Permit` extension point, implemented by the `GangScheduling` plugin,
is used to determine whether the `minCount` constraint is satisfied during that cycle.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When putting on a hat of somebody without the WAS expertise and reading this bullet point as it is right now, it's unclear to me what constraint exactly is being verified in the Permit extension point.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed this sentence, PTAL

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds better now, thanks!

Comment thread content/en/docs/concepts/scheduling-eviction/podgroup-scheduling.md Outdated
@tosi3k
Copy link
Copy Markdown
Member

tosi3k commented Apr 2, 2026

/assign

@macsko macsko force-pushed the add_docs_for_workload_cycle branch from 3bbb44b to 97e6a04 Compare April 2, 2026 15:04
@tosi3k
Copy link
Copy Markdown
Member

tosi3k commented Apr 2, 2026

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 2, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: 98a4b46451fb71174a4bda8a8120023e44632ced

@macsko macsko force-pushed the add_docs_for_workload_cycle branch from 97e6a04 to 4e741f8 Compare April 3, 2026 10:40
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 3, 2026
@k8s-ci-robot k8s-ci-robot requested a review from tosi3k April 3, 2026 10:40
@macsko
Copy link
Copy Markdown
Member Author

macsko commented Apr 3, 2026

/assign @lmktfy
For docs approval

Copy link
Copy Markdown
Contributor

@tengqm tengqm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@k8s-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tengqm

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 5, 2026
@tosi3k
Copy link
Copy Markdown
Member

tosi3k commented Apr 7, 2026

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 7, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: 2a320b84edcf5aee0df797fb00e30839af7a7824

@k8s-ci-robot k8s-ci-robot merged commit 63d4591 into kubernetes:dev-1.36 Apr 7, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from Needs Triage to Done in SIG Scheduling Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/workload-aware Categorizes an issue or PR as relevant to Workload-aware and Topology-aware scheduling subprojects. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/scheduling Categorizes an issue or PR as relevant to SIG Scheduling. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants