-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Promote Topology Labels to GA #837
Comments
related #793 |
/sig cloud-provider |
/kind api-change related discussion in kubernetes/kubernetes#73616 (comment) |
@MorrisLaw has volunteered to implement this once the KEP PR is merged, thank you! |
Don't think this will make it to v1.14 /milestone v1.15 |
Hi @andrewsykim. I'm the 1.15 Enhancement Lead. Has the KEP for this been approved? If so, can we update the original post to reflect it? We will get this tracked for 1.15. If there are any open k/k PRs, please drop them into this issue so they can be properly tracked during code freeze. Thanks! /stage stable |
/assign @andrewsykim |
@andrewsykim Kubernetes 1.15 Enhancement Freeze is 4/30/2019. To be included in the Kubernetes 1.15 milestone, KEPs are required to be merged and in an "Implementable" state with proper test plans and graduation criteria. Please submit any PRs needed to make this KEP adhere to inclusion criteria. If this will slip from the 1.15 milestone, please let us know so we can make appropriate tracking changes. |
@andrewsykim Enhancement Freeze for Kubernetes 1.15 has passed and this did not meet the deadline. This is now being removed from the 1.15 milestone and the tracking sheet. If there is a need for this to be in 1.15, please file an Enhancement Exception. Thank you. |
Hi @andrewsykim , I'm the 1.16 Enhancement Lead. Is this feature going to be graduating alpha/beta/stable stages in 1.16? Please let me know so it can be added to the 1.16 Tracking Spreadsheet. If not's graduating, I will remove it from the milestone and change the tracked label. Once coding begins or if it already has, please list all relevant k/k PRs in this issue so they can be tracked properly. Milestone dates are Enhancement Freeze 7/30 and Code Freeze 8/29. Thank you. |
Andrew - is this implemented somewhere just waiting for review? Or is it stalled? |
Implemented here kubernetes/kubernetes#81431 but PR slipped my radar in the last week. Will brush it up some time today and assign it to you :) |
Hi @andrewsykim Just touching base as you said that the plan was to remove the beta labels for v1.21. The KEP still says the same, but just wanted to ping in case this changed, as the Enhancements Freeze is October 6th and the KEP is missing test plans. If the 1.21 target has changed, please let me know, otherwise we will keep tracking this as deferred til 1.21. Best, |
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 |
Hi @andrewsykim, 1.21 Enhancements Lead here. I'm clearing the milestone to reflect that this enhancement was not entered into the tracking sheet before the enhancements freeze, therefore not opting into the 1.21 release. /milestone clear |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
/remove-lifecycle stale |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close |
@k8s-triage-robot: 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. |
Enhancement Description
The topology labels by cloud providers we apply on nodes have been in "beta" for a very long time. Given their usage across the project we should GA them.
The 3 labels are:
beta.kubernetes.io/instance-type
failure-domain.beta.kubernetes.io/zone
failure-domain.beta.kubernetes.io/region
They should be renamed to:
node.kubernetes.io/instance-type
topology.kubernetes.io/zone
topology.kubernetes.io/region
While keeping backwards compatibility in mind. Changing the label format may also be an option given we have to rename the label key anyways.
Kubernetes Enhancement Proposal: https://github.com/kubernetes/enhancements/blob/master/keps/sig-cloud-provider/20190215-promoting-cloud-provider-labels.md
Primary contact (assignee):
@andrewsykim
Responsible SIGs:
SIG Cloud Provider, SIG Node
Enhancement target (which target equals to which milestone):
The text was updated successfully, but these errors were encountered: