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

Conform worker nodes powered by spot fleets to the ones powered by autoscaling groups #167

Merged
merged 3 commits into from Dec 16, 2016

Conversation

mumoshu
Copy link
Contributor

@mumoshu mumoshu commented Dec 15, 2016

The last two TODOs mentioned in #112 are addressed

@mumoshu mumoshu added this to the v0.9.3-rc.1 milestone Dec 15, 2016
@codecov-io
Copy link

codecov-io commented Dec 15, 2016

Current coverage is 69.36% (diff: 100%)

Merging #167 into master will not change coverage

@@             master       #167   diff @@
==========================================
  Files             4          4          
  Lines          1126       1126          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits            781        781          
  Misses          259        259          
  Partials         86         86          

Powered by Codecov. Last update b918f1e...c61b0c5

… spot fleet

to conform these nodes to the ones powered by a autoscaling group

ref kubernetes-retired#112
see also http://docs.aws.amazon.com/cli/latest/reference/ec2/create-tags.html and http://stackoverflow.com/a/1250279 for implementation details
@mumoshu
Copy link
Contributor Author

mumoshu commented Dec 16, 2016

From now on, both ASG-based and SpotFleet-based worker nodes can be added to an ELB load balancer with cluster.yaml:

experimental:
  loadBalancer:
    names: [ "name-of-lb" ]

image

kubeawstest9-kube-aws-workers are coming from an ASG in a main cluster. Likewise, kubeawstest9-nodepool1-kube-aws-workers are coming from a spot fleet in a node pool.

@mumoshu
Copy link
Contributor Author

mumoshu commented Dec 16, 2016

Also from now on, both ASG-based and SpotFleet-based worker nodes can be tagged with Name and kube-aws:node-pool:name by default.

image

@mumoshu
Copy link
Contributor Author

mumoshu commented Dec 16, 2016

Updated E2E test suite to cover deployments to existing VPCs 🎉

@mumoshu mumoshu merged commit bd6d62a into kubernetes-retired:master Dec 16, 2016
@mumoshu mumoshu deleted the conform-spot-fleet-to-asg branch December 16, 2016 09:52
kylehodgetts pushed a commit to HotelsDotCom/kube-aws that referenced this pull request Mar 27, 2018
…leet-to-asg

Conform worker nodes powered by spot fleets to the ones powered by autoscaling groups
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.

None yet

2 participants