-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
aws-stepfunction: Failed to create correct IAM role for step function #33580
Comments
The issue is in the configurePolicyStatements() method of the BatchSubmitJob class. Unlike other task implementations, it doesn't check if jobQueueArn is a JsonPath expression. The fix would be to modify this method to check if jobQueueArn is a JsonPath expression, and if so, use a wildcard (*) instead:
|
Do we have any plans to merge this PR recently? |
…3670) ### Issue # (if applicable) Closes #33580 ### Reason for this change Incorrect IAM Policy for jobQueueArn when using JsonPath or JSONata ### Description of changes For JsonPath or JSONata `jobQueueArn`, IAM Policy use wildcard (*) ### Describe any new or updated permissions being added For JsonPath or JSONata `jobQueueArn`, IAM Policy use wildcard (*) ### Description of how you validated changes Unit test ### Checklist - [x] 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*
Comments on closed issues and PRs are hard for our team to see. |
Describe the bug
When trying the code here:
I am just building a step function to submit a batch job with dynamic parameter values by using payload during submit time.
However, when deploying the resource, the code above is trying to create one IAM role with following resources:
$.inferenceJob.jobQueueArn cannot be retrieved so it will lead to an error. How to handle this case?
Regression Issue
Last Known Working CDK Version
2.1000.2
Expected Behavior
I assume the role will be deployed correctly. Even if it did not add correct rules, at least it deployed the rules successfully.
Current Behavior
It will throw an error saying:
Reproduction Steps
Copy the code above and deploy and it will see the error.
Possible Solution
Anyway we can ban the step function cdk to create the role itself? We can use our own rules this way.
Additional Information/Context
No response
CDK CLI Version
2.1000.2
Framework Version
No response
Node.js Version
21
OS
macos
Language
Python
Language Version
3.9
Other information
No response
The text was updated successfully, but these errors were encountered: