Skip to content

Commit

Permalink
fix(stepfunctions-tasks): stateMachine construct doesn't generate a v…
Browse files Browse the repository at this point in the history
…alid policy for default StateMachineRole (#31801)

### Issue # (if applicable)

Closes #31714.

### Reason for this change

Currently, the step functions `runEcsTask()` will create an IAM policy. The `Resource` section is an ARN constructed by CDK with wildcard `*` appending at the end. However, CDK should `Ref` the resource directly instead of constructing the ARN, while keeping the revision number.

### Description of changes

The same solution as #31615. However this change needs to behind a feature flag because it could be a breaking change.

### Description of how you validated changes

Integration test. Also checked the synth template.

### Checklist
- [ ] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
xazhao authored Oct 18, 2024
1 parent 8208774 commit efbbddb
Show file tree
Hide file tree
Showing 16 changed files with 4,002 additions and 4 deletions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit efbbddb

Please sign in to comment.