fix(aws-ecs): use executionRole for event rule target

Fixes aws#2015.
SoManyHs · Apr 2, 2019 · acd2de1 · acd2de1
1 parent 997dbcc
commit acd2de1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/packages/@aws-cdk/aws-ecs/lib/ec2/ec2-event-rule-target.ts b/packages/@aws-cdk/aws-ecs/lib/ec2/ec2-event-rule-target.ts
@@ -95,7 +95,7 @@ export class Ec2EventRuleTarget extends cdk.Construct implements events.IEventRu
     //
     // It never needs permissions to the Task Role.
     if (this.taskDefinition.executionRole !== undefined) {
-      this.taskDefinition.taskRole.grantPassRole(this.eventsRole);
+      this.taskDefinition.executionRole.grantPassRole(this.eventsRole);
     }
   }
-}
+}
diff --git a/packages/@aws-cdk/aws-ecs/test/ec2/integ.event-task.lit.expected.json b/packages/@aws-cdk/aws-ecs/test/ec2/integ.event-task.lit.expected.json
@@ -491,8 +491,8 @@
         "Action": "lambda:InvokeFunction",
         "FunctionName": {
           "Fn::GetAtt": [
-             "EcsClusterDefaultAutoScalingGroupDrainECSHookFunctionE17A5F5E",
-             "Arn"
+            "EcsClusterDefaultAutoScalingGroupDrainECSHookFunctionE17A5F5E",
+            "Arn"
           ]
         },
         "Principal": "sns.amazonaws.com",
@@ -912,7 +912,7 @@
               "Effect": "Allow",
               "Resource": {
                 "Fn::GetAtt": [
-                  "TaskDefTaskRole1EDB4A67",
+                  "TaskDefExecutionRoleB4775C97",
                   "Arn"
                 ]
               }
@@ -1163,4 +1163,4 @@
       "Description": "S3 key for asset version \"aws-ecs-integ-ecs/AdoptEcrRepositorydbc60defc59544bcaa5c28c95d68f62c/Code\""
     }
   }
-}
+}