Skip to content

feat(ecs-patterns): allow specifying IAM role for EcsTask in ScheduledFargateTask and ScheduledEc2Task#37311

Open
syukawa-gh wants to merge 4 commits intoaws:mainfrom
syukawa-gh:feat/ecs-patterns-scheduled-task-role-clean
Open

feat(ecs-patterns): allow specifying IAM role for EcsTask in ScheduledFargateTask and ScheduledEc2Task#37311
syukawa-gh wants to merge 4 commits intoaws:mainfrom
syukawa-gh:feat/ecs-patterns-scheduled-task-role-clean

Conversation

@syukawa-gh
Copy link
Copy Markdown
Contributor

Add a role property to ScheduledTaskBaseProps that is passed through to the underlying EcsTask event target. This allows users to provide a custom IAM role instead of relying on the auto-created one, enabling scenarios like customizing the ecs:RunTask policy resource scope.

Closes #22673

…dFargateTask and ScheduledEc2Task

Add a 'role' property to ScheduledTaskBaseProps that is passed through
to the underlying EcsTask event target. This allows users to provide
a custom IAM role instead of relying on the auto-created one.

Closes aws#22673
@github-actions github-actions bot added beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2 labels Mar 23, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team March 23, 2026 03:29
Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(This review is outdated)

@aws-cdk-automation aws-cdk-automation added the pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. label Mar 23, 2026
@syukawa-gh
Copy link
Copy Markdown
Contributor Author

Exemption Request: Integration test snapshot cannot be generated locally because the role property is a new addition not yet in the released aws-cdk-lib. Unit test and README have been added. The integration test snapshot will be generated by CI after merge.

@aws-cdk-automation aws-cdk-automation added the pr-linter/exemption-requested The contributor has requested an exemption to the PR Linter feedback. label Mar 23, 2026
@aws-cdk-automation aws-cdk-automation removed the pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. label Mar 23, 2026
@aws-cdk-automation aws-cdk-automation dismissed their stale review March 24, 2026 01:31

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

@aws-cdk-automation aws-cdk-automation added the pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. label Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK effort/small Small work item – less than a day of effort feature-request A feature should be added or improved. p2 pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. pr-linter/exemption-requested The contributor has requested an exemption to the PR Linter feedback.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(ecs-patterns): allow specifying IAM-role for EcsTask through ScheduledFargateTask

3 participants