-
Notifications
You must be signed in to change notification settings - Fork 39k
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
Deprecate Node.Spec.Unschedulable when TaintNodeByCondition is GA'ed #69010
Comments
/xref #68984 |
hi~ I am interested in this job, and commit a pr to give it a try. Would you please take a look to check if I do it right? :) |
@tianshapjq Thanks for the PR, but as it is mentioned in the issue, we should deprecate Node.Spec.Unschedulable only when TaintNodeByCondition is GA. TaintNodeByCondition is beta in 1.12 and will probably not be in GA in 1.13. So, there will be some time before we can deprecate Node.Spec.Unschedulable. |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
/remove-lifecycle stale |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Stale issues rot after 30d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
Rotten issues close after 30d of inactivity. Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
@fejta-bot: Closing this issue. In response to this:
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. |
/reopen |
@draveness: Reopened this issue. In response to this:
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. |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
/remove-lifecycle stale |
We can't deprecate the Unschedulable field unless we have v2 node API, per discussion in #84266 (comment). We'll revisit this issue after that /lifecycle frozen |
/close feel free to reopen this |
@draveness: Closing this issue. In response to this:
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. |
TaintNodeByCondition has replaced the existing node condition. With this feature node conditions that affect feasibility of nodes are reflected as taints on nodes. This feature is promoted to Beta in 1.12. We should deprecate node conditions once the feature is GA'ed in future releases.
/sig scheduling
/sig node
The text was updated successfully, but these errors were encountered: