Skip to content

Conversation

@helayoty
Copy link
Member

@helayoty helayoty commented Oct 20, 2025

Description

Update docs for extend tolerations operators, Gt and Lt, for graduation to alpha.

Issue

KEP Issue: link
KEP: link

/sig scheduling

Issue

Closes: #

@k8s-ci-robot k8s-ci-robot added this to the 1.35 milestone Oct 20, 2025
@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. sig/apps Categorizes an issue or PR as relevant to SIG Apps. sig/scheduling Categorizes an issue or PR as relevant to SIG Scheduling. labels Oct 20, 2025
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Oct 20, 2025
@netlify
Copy link

netlify bot commented Oct 20, 2025

👷 Deploy Preview for kubernetes-io-vnext-staging processing.

Name Link
🔨 Latest commit ded370a
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-vnext-staging/deploys/691cbc3a68ce6d0008f8a9f6

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign tengqm for approval. For more information see the Code Review Process.

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

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 language/en Issues or PRs related to English language label Oct 20, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in SIG Apps Oct 20, 2025
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Oct 20, 2025
@netlify
Copy link

netlify bot commented Oct 20, 2025

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit ded370a
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/691cbc3a4b2640000805fa4b
😎 Deploy Preview https://deploy-preview-52824--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.

Signed-off-by: Heba Elayoty <heelayot@microsoft.com>
@helayoty helayoty force-pushed the helayoty/5471-extend-tolerations-alpha branch from b19d7a4 to b12c545 Compare October 20, 2025 22:32
Copy link
Member

@lmktfy lmktfy left a comment

Choose a reason for hiding this comment

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

Thanks for the PR.

I know this is a draft, but I want to provide some early feedback. I am concerned that it might not be right for our docs once you finish it, and I hope that the early feedback helps you to find the right way to tweak it.

Signed-off-by: Heba Elayoty <heelayot@microsoft.com>
@helayoty helayoty requested a review from lmktfy October 22, 2025 02:59
@helayoty
Copy link
Member Author

@lmktfy Many thanks for the early feedback. Addressed all and appreciate if you have time for another round.

@helayoty helayoty moved this to Backlog in SIG Scheduling Oct 22, 2025
@Urvashi0109
Copy link
Contributor

Hello @helayoty 👋, v1.35 Docs Team here again!

We are closing in on the deadline to get your PR ready for review before Tuesday 18th November 2025, so I'm sending a second reminder. Please take a look at the Documenting for a release - PR Ready for Review document to get your PR ready for review before the deadline.

Please also let us know once your PR is fully Ready for Review -- meaning all documentation updates are complete and it's awaiting reviewer feedback -- so we can update our tracking.

Thank you!

@helayoty helayoty marked this pull request as ready for review November 17, 2025 18:22
@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 Nov 17, 2025
@helayoty helayoty moved this from Backlog to Needs Review in SIG Scheduling Nov 18, 2025
Signed-off-by: Heba Elayoty <heelayot@microsoft.com>
@helayoty
Copy link
Member Author

Please also let us know once your PR is fully Ready for Review -- meaning all documentation updates are complete and it's awaiting reviewer feedback -- so we can update our tracking.

@Urvashi0109 , PR is ready for review.


For numeric operators, both the toleration and taint values must be valid integers.
If either value cannot be parsed as an integer, the toleration does not match.
For numeric operators, both the toleration and taint values must be valid integers. If either value cannot be parsed as an integer, the toleration does not match.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why change this?
We encourage wrapping long "lines" for ease of reading and maintenance.

Copy link
Member Author

Choose a reason for hiding this comment

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

Addressed.

validates that the toleration values are valid integers. Taint values on nodes are not
validated at node registration time. If a node has a non-numeric taint value
(for example, `servicelevel.organization.example/agreed-service-level=high:NoSchedule`), pods with numeric comparison
When you create a Pod that uses `Gt` or `Lt` tolerations operators, the API server validates that the toleration values are valid integers. Taint values on nodes are not validated at node registration time. If a node has a non-numeric taint value (for example, `servicelevel.organization.example/agreed-service-level=high:NoSchedule`), pods with numeric comparison
Copy link
Contributor

Choose a reason for hiding this comment

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

Please check https://sembr.org/ for the convention we prefer.

Copy link
Member Author

Choose a reason for hiding this comment

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

Addressed.

Signed-off-by: Heba Elayoty <heelayot@microsoft.com>
@helayoty helayoty requested a review from tengqm November 18, 2025 18:35
@sayanchowdhury
Copy link
Member

Hello @helayoty 👋! I'm reaching out from the Docs team. Just checking in as we approach Docs Freeze on 3rd December 2025, 12:00 UTC.

This documentation appears to still be under review. To meet the Docs Freeze, this PR must have a technical review as well as lgtm and approve labels applied, without any unaddressed comments or concerns from SIG Docs. The status of this enhancement is marked as at risk for docs freeze. Thank you!

registration time. If a node has a non-numeric taint value (for example,
`servicelevel.organization.example/agreed-service-level=high:NoSchedule`),
pods with numeric comparison operators will not match that taint and cannot schedule on that node.
{{< /note >}}
Copy link
Member

@lmktfy lmktfy Nov 21, 2025

Choose a reason for hiding this comment

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

nit: could mention that people can use admission-time checks to block node registrations with invalid taints.

It's a very strict choice but you can do it if you need to.

* Pods that do not tolerate the taint are evicted immediately
* Pods that tolerate the taint without specifying `tolerationSeconds` in

* Pods that do not tolerate the taint are evicted immediately
Copy link
Member

Choose a reason for hiding this comment

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

Why the indentation change,?

@helayoty helayoty requested a review from lmktfy November 24, 2025 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language sig/apps Categorizes an issue or PR as relevant to SIG Apps. 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

Status: Needs Triage
Status: Needs Review

Development

Successfully merging this pull request may close these issues.

6 participants