Skip to content

Commit

Permalink
Update packages/@aws-cdk/aws-eks/lib/managed-nodegroup.ts
Browse files Browse the repository at this point in the history
Co-Authored-By: Elad Ben-Israel <[email protected]>
  • Loading branch information
pahud and Elad Ben-Israel authored Mar 17, 2020
1 parent f060878 commit c5e77b6
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions packages/@aws-cdk/aws-eks/lib/managed-nodegroup.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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, {
Expand Down

0 comments on commit c5e77b6

Please sign in to comment.