Skip to content
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

AKS support for node labels as part of agentPoolProfile to persist labels through upgrade/scale #1088

Closed
vflopes opened this issue Jul 3, 2019 · 21 comments

Comments

@vflopes
Copy link

vflopes commented Jul 3, 2019

What happened: Can't put labels on node

What you expected to happen: A documentation or guide explaining how to define labels for nodes trough nodepools

Anything else we need to know?: I'm using az aks commands to manage and deploy AKS clusters.

Environment:

  • Kubernetes version: 1.12.8
  • Size of cluster (how many worker nodes are in the cluster?): From one to any size (we're using cluster autoscaler)
  • General description of workloads in the cluster: Services exposing HTTP APIs

I'm deploying AKS clusters and I want to split the workloads by tiers and to have this behavior I'm using nodeSelectors on my Deployments, but to make this work I need to define which labels will be assigned to new nodes created by the autoscaler. I'm not seeing this feature anywhere (Microsoft/Azure/AKS docs), are there any command or API where I can define the template (and the labels) for new nodes created by the autoscaler?

@Vitorspk
Copy link

Vitorspk commented Jul 3, 2019

I have the same problem when trying to scale nodes

@jnoller
Copy link
Contributor

jnoller commented Jul 12, 2019

This is on the roadmap for 2019. Adding to in progress in https://github.com/Azure/AKS/projects

@MacTynow
Copy link

MacTynow commented Sep 2, 2019

Hi @jluk (reposting from terraform azure rm provider) , I'm confused because I can see references to custom labels in https://github.com/Azure/aks-engine/blob/master/examples/kubernetes-labels/kubernetes.json as well as https://github.com/Azure/aks-engine/blob/8f5de6d71f67c1b3f618338626f7e86a8ea98727/pkg/api/types.go#L547.

Do you mean that this doesn't work yet or that the behavior is different from using --kubelet-extra-args '--node-labels=foo=bar?

@jluk jluk changed the title Define node's labels through nodepools AKS support for node labels as part of agentPoolProfile to persist labels through upgrade/scale Sep 3, 2019
@jluk
Copy link
Contributor

jluk commented Sep 3, 2019

@MacTynow that would be the AKS Engine project which is separate from AKS. The label functionality you see as part of the agentpool is what we plan to bring into AKS, but until we do you won't have labels reapplied through upgrade/scale.

@Benjiiim
Copy link

@jluk, would you mind to provide an update on this one? Thanks a lot.

@jluk
Copy link
Contributor

jluk commented Sep 18, 2019

Hey @Benjiiim we'll get it before end of the year, that's my best estimate right now as we're wrapping up some important behavior for agent pools wrt quality of servicing and support of older/existing clusters. I should have a better estimate of timing in 4-6 weeks.

@jluk jluk added the nodepools label Oct 2, 2019
@dazdaz
Copy link

dazdaz commented Oct 18, 2019

@jluk any update on the timeline for when this will be implemented ? This feature would be useful right now. Thanks a lot.

@jluk
Copy link
Contributor

jluk commented Oct 18, 2019

This is scoping for December, as we'll need to release a new API version.

@ncole
Copy link

ncole commented Nov 5, 2019

Is this still slated for December?

@jluk
Copy link
Contributor

jluk commented Nov 5, 2019

As of today, this is still on plan by end of December. This is subject to change but will update this issue if it does.

@jayush
Copy link

jayush commented Nov 5, 2019

Thanks @jluk for the update. Will this feature be released as a preview feature or GA feature in December?

@dstockton
Copy link

@jluk do you have any update, please? Is it still looking like the end of Dec for this?
Many thanks, David

@jluk
Copy link
Contributor

jluk commented Dec 6, 2019

Hey folks, thanks for bumping this. Labels support has been delayed a bit, looking at early in the new year for next ETA. We have just closed on setting agent pool tag support to be released by end of year, but rest assured this request for agent pool labels is firmly on the roadmap and will be delivered.

@michael-burt
Copy link

Can we get an update on this feature?

@ncole
Copy link

ncole commented Jan 16, 2020

I checked the Azure SDK for Go and at the time of commenting it looks like v38.2.0 contains this functionality (possibly earlier versions, but I only checked that version).

@petegrimsdale
Copy link

@jluk Is there an estimate as to when this capability will be available?

@jluk
Copy link
Contributor

jluk commented Feb 18, 2020

This is supported via API now. We are closing on the CLI support and documentation.

This requires operations on the target agentPool via the agentPool API. These can only be set on creation, mutation is not supported in this release.

sagikazarmark added a commit to banzaicloud/pipeline that referenced this issue Feb 22, 2020
Azure AKS now supports adding initial node labels to agent pools.
Use it instead of manually adding labels.

Azure/AKS#1088 (comment)
sagikazarmark added a commit to banzaicloud/pipeline that referenced this issue Feb 23, 2020
Azure AKS now supports adding initial node labels to agent pools.
Use it instead of manually adding labels.

Azure/AKS#1088 (comment)
@jluk
Copy link
Contributor

jluk commented Mar 11, 2020

This capability is now shipped and available on the latest CLI build, az v2.2.0.

https://github.com/Azure/azure-cli/releases/tag/azure-cli-2.2.0

@michael-burt
Copy link

Thanks @jluk you are doing a great job of communicating this stuff and we appreciate it.

@jluk
Copy link
Contributor

jluk commented Mar 11, 2020

Thanks a lot for the kind words, @michael-burt glad this is helpful!

@jcjwent
Copy link

jcjwent commented Mar 27, 2020

Hello @jluk thanks a lot for releasing this feature. Have been waiting for this.

I am creating my node pools using the ContainerServices Fluent SDK. The AgentPools.CreateOrUpdate operation uses the AgentPoolInner class documented here. Do you have a suggestion how to add node labels to the node pool added using this operation? The AgentPoolInner class does not seem to have a provision for this.

Hope you can guide me to a solution.

@ghost ghost locked as resolved and limited conversation to collaborators Jul 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Development

No branches or pull requests