From c5e77b6bb63ee3b269032d9608e6be0347c6ce0d Mon Sep 17 00:00:00 2001 From: Pahud Hsieh Date: Wed, 18 Mar 2020 07:28:10 +0800 Subject: [PATCH] Update packages/@aws-cdk/aws-eks/lib/managed-nodegroup.ts Co-Authored-By: Elad Ben-Israel --- packages/@aws-cdk/aws-eks/lib/managed-nodegroup.ts | 2 -- 1 file changed, 2 deletions(-) diff --git a/packages/@aws-cdk/aws-eks/lib/managed-nodegroup.ts b/packages/@aws-cdk/aws-eks/lib/managed-nodegroup.ts index 6adf67496b7e5..6250b82812a14 100644 --- a/packages/@aws-cdk/aws-eks/lib/managed-nodegroup.ts +++ b/packages/@aws-cdk/aws-eks/lib/managed-nodegroup.ts @@ -259,8 +259,6 @@ export class Nodegroup extends Resource implements INodegroup { // do not attempt to map the role if `kubectl` is not enabled for this // cluster or if `mapRole` is set to false. By default this should happen. - // const mapRole = options.mapRole === undefined ? true : options.mapRole; - // if (mapRole && this.kubectlEnabled) { if (this.cluster.kubectlEnabled) { // see https://docs.aws.amazon.com/en_us/eks/latest/userguide/add-user-role.html this.cluster.awsAuth.addRoleMapping(ngRole, {