Merge pull request #25 from cmdlabs/fix-autoscaler-tags

Fix autoscaler tags
cmdlabs · Jul 4, 2019 · f442049 · f442049
2 parents e3ae766 + 3eea2ad
commit f442049
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,7 +4,9 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [0.6.1] - 2019-06-20
+## [0.6.1] - 2019-07-04
+### Fixed
+- Resolved issue that prevented cluster-autoscaler from autoscaling workers.
 ### Added
 - Added ec2:DescribeNetworkInterfaces to alb-ingress policy
 

diff --git a/kiam_workers.tf b/kiam_workers.tf
@@ -54,7 +54,7 @@ resource "aws_autoscaling_group" "kiam_workers" {
         "propagate_at_launch" = "true"
       },
       {
-        "key"                 = "k8s.io/cluster-autoscaler/${var.kiam_autoscaling_enabled == 1 ? "enabled" : "disabled"}"
+        "key"                 = "k8s.io/cluster-autoscaler/${var.kiam_autoscaling_enabled == true ? "enabled" : "disabled"}"
         "value"               = "true"
         "propagate_at_launch" = "false"
       },

diff --git a/workers.tf b/workers.tf
@@ -54,7 +54,7 @@ resource "aws_autoscaling_group" "workers" {
         "propagate_at_launch" = "true"
       },
       {
-        "key"                 = "k8s.io/cluster-autoscaler/${lookup(var.workers[count.index], "autoscaling_enabled", true) == 1 ? "enabled" : "disabled"}"
+        "key"                 = "k8s.io/cluster-autoscaler/${lookup(var.workers[count.index], "autoscaling_enabled", true) == true ? "enabled" : "disabled"}"
         "value"               = "true"
         "propagate_at_launch" = "false"
       },