fix(eks): incorrect enableDockerBridge value when enabled (#8895)

When `enableDockerBridge` is enabled in `BootstrapOptions`, pass the value correctly to user-data. Fixes: #5786 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
aws · Jul 5, 2020 · ea0552a · ea0552a
1 parent a732d14
commit ea0552a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/@aws-cdk/aws-eks/lib/user-data.ts b/packages/@aws-cdk/aws-eks/lib/user-data.ts
@@ -20,7 +20,7 @@ export function renderAmazonLinuxUserData(clusterName: string, autoScalingGroup:
   }
 
   if (options.enableDockerBridge) {
-    extraArgs.push('--enable-docker-bridge');
+    extraArgs.push('--enable-docker-bridge true');
   }
 
   if (options.dockerConfigJson) {
@@ -67,4 +67,4 @@ export enum LifecycleLabel {
    * spot instances
    */
   SPOT = 'Ec2Spot'
-}
+}
diff --git a/packages/@aws-cdk/aws-eks/test/test.user-data.ts b/packages/@aws-cdk/aws-eks/test/test.user-data.ts
@@ -90,7 +90,7 @@ export = {
     }));
 
     // THEN
-    test.deepEqual(userData[1], '/etc/eks/bootstrap.sh my-cluster-name --kubelet-extra-args "--node-labels lifecycle=OnDemand" --use-max-pods true --enable-docker-bridge');
+    test.deepEqual(userData[1], '/etc/eks/bootstrap.sh my-cluster-name --kubelet-extra-args "--node-labels lifecycle=OnDemand" --use-max-pods true --enable-docker-bridge true');
     test.done();
   },
-Original file line number
+Diff line change
@@ Expand Up / @@ -90,7 +90,7 @@ export = { @@
         }));
         // THEN
-        test.deepEqual(userData[1], '/etc/eks/bootstrap.sh my-cluster-name --kubelet-extra-args "--node-labels lifecycle=OnDemand" --use-max-pods true --enable-docker-bridge');
+        test.deepEqual(userData[1], '/etc/eks/bootstrap.sh my-cluster-name --kubelet-extra-args "--node-labels lifecycle=OnDemand" --use-max-pods true --enable-docker-bridge true');
         test.done();
       },
@@ Expand Down @@