Skip to content

Commit 3b07346

Browse files
authored
fix(stepfunctions-tasks): SageMakerCreateTrainingJob add sagemaker:AddTags permission (#32536)
### Issue # (if applicable) Closes #32294 ### Reason for this change - Missing permission <img width="722" alt="Screenshot 2024-12-15 171605" src="https://github.com/user-attachments/assets/c5626c58-fef7-48bf-afd1-934978ce7baa" /> ### Description of changes - StepFunctions add `sagemaker:AddTags` permission ### Description of how you validated changes Unit + Integration 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*
1 parent 862072e commit 3b07346

File tree

28 files changed

+33964
-60
lines changed

28 files changed

+33964
-60
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
__label__4 linux ready for prime time , intel says , despite all the linux hype , the open-source movement has yet to make a huge splash in the desktop market . that may be about to change , thanks to chipmaking giant intel corp .
2+
__label__2 bowled by the slower one again , kolkata , november 14 the past caught up with sourav ganguly as the indian skippers return to international cricket was short lived .

packages/@aws-cdk-testing/framework-integ/test/aws-stepfunctions-tasks/test/sagemaker/integ.call-sagemaker.js.snapshot/aws-stepfunctions-integ-sagemaker.assets.json

+3-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-stepfunctions-tasks/test/sagemaker/integ.call-sagemaker.js.snapshot/aws-stepfunctions-integ-sagemaker.template.json

+7-6
Original file line numberDiff line numberDiff line change
@@ -282,6 +282,7 @@
282282
"Statement": [
283283
{
284284
"Action": [
285+
"sagemaker:AddTags",
285286
"sagemaker:CreateTrainingJob",
286287
"sagemaker:DescribeTrainingJob",
287288
"sagemaker:StopTrainingJob"
@@ -447,12 +448,6 @@
447448
"StateMachine2E01A3A5": {
448449
"Type": "AWS::StepFunctions::StateMachine",
449450
"Properties": {
450-
"RoleArn": {
451-
"Fn::GetAtt": [
452-
"StateMachineRoleB840431D",
453-
"Arn"
454-
]
455-
},
456451
"DefinitionString": {
457452
"Fn::Join": [
458453
"",
@@ -518,6 +513,12 @@
518513
":states:::sagemaker:updateEndpoint\",\"Parameters\":{\"EndpointConfigName.$\":\"$.Endpoint.Config\",\"EndpointName.$\":\"$.Endpoint.Name\"}}}}"
519514
]
520515
]
516+
},
517+
"RoleArn": {
518+
"Fn::GetAtt": [
519+
"StateMachineRoleB840431D",
520+
"Arn"
521+
]
521522
}
522523
},
523524
"DependsOn": [

packages/@aws-cdk-testing/framework-integ/test/aws-stepfunctions-tasks/test/sagemaker/integ.call-sagemaker.js.snapshot/cdk.out

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-stepfunctions-tasks/test/sagemaker/integ.call-sagemaker.js.snapshot/integ.json

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-stepfunctions-tasks/test/sagemaker/integ.call-sagemaker.js.snapshot/manifest.json

+3-2
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/@aws-cdk-testing/framework-integ/test/aws-stepfunctions-tasks/test/sagemaker/integ.call-sagemaker.js.snapshot/tree.json

+40-39
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)