Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

Experimental feature: User-specified node labels for worker nodes #149

Merged

Conversation

mumoshu
Copy link
Contributor

@mumoshu mumoshu commented Dec 10, 2016

This complements Node Pools(#46) and Spot Fleet support(#112)

The former experimental.nodeLabel configuration key is renamed to experimental.awsNodeLabels to avoid collision with newly added experimental.nodeLabels and consistency with experimental.awsEnvironment.

cc @pieterlange @c-knowles

@codecov-io
Copy link

codecov-io commented Dec 10, 2016

Current coverage is 69.90% (diff: 100%)

Merging #149 into master will increase coverage by 0.21%

@@             master       #149   diff @@
==========================================
  Files             4          4          
  Lines          1095       1103     +8   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits            763        771     +8   
  Misses          248        248          
  Partials         84         84          

Powered by Codecov. Last update 3395dfc...f2beccf

@mumoshu
Copy link
Contributor Author

mumoshu commented Dec 10, 2016

@pieterlange I'm willing to rename the former NodeLabel to AwsNodeLabels as I've explained in the description of this PR. Any thoughts on the move and/or naming?

The former NodeLabel is added in v0.9.2-rc.1. I'd rather like to merge this before we cut 0.9.2 final so it won't be a breaking change.

This complements Node Pools(kubernetes-retired#46) and Spot Fleet support(kubernetes-retired#112)

The former `experimental.nodeLabel` configuration key is renamed to `experimental.awsNodeLabels` to avoid collision with newly added `experimental.nodeLabels` and consistency with `experimental.awsEnvironment`.
@mumoshu mumoshu added this to the v0.9.2-rc.5 milestone Dec 14, 2016
@mumoshu mumoshu merged commit fd97e01 into kubernetes-retired:master Dec 14, 2016
@mumoshu mumoshu deleted the user-specified-node-labels branch December 14, 2016 06:01
kylehodgetts pushed a commit to HotelsDotCom/kube-aws that referenced this pull request Mar 27, 2018
…-node-labels

Experimental feature: User-specified node labels for worker nodes
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants