Skip to content

Commit 618a58e

Browse files
bchamppseshubaws
andauthored
feat: use managed policy for Durable Functions (#3854)
Co-authored-by: seshubaws <116689586+seshubaws@users.noreply.github.com>
1 parent 2959621 commit 618a58e

10 files changed

+23
-19
lines changed

samtranslator/model/sam_resources.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -789,7 +789,11 @@ def _construct_role(
789789
else IAMRolePolicies.lambda_assume_role_policy()
790790
)
791791

792-
managed_policy_arns = [ArnGenerator.generate_aws_managed_policy_arn("service-role/AWSLambdaBasicExecutionRole")]
792+
managed_policy_arns = (
793+
[ArnGenerator.generate_aws_managed_policy_arn("service-role/AWSLambdaBasicDurableExecutionRolePolicy")]
794+
if self.DurableConfig
795+
else [ArnGenerator.generate_aws_managed_policy_arn("service-role/AWSLambdaBasicExecutionRole")]
796+
)
793797

794798
tracing = intrinsics_resolver.resolve_parameter_refs(self.Tracing)
795799

tests/translator/output/aws-cn/function_with_durable_config.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
"Version": "2012-10-17"
4747
},
4848
"ManagedPolicyArns": [
49-
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
49+
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
5050
],
5151
"Tags": [
5252
{

tests/translator/output/aws-cn/function_with_durable_config_globals.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
"Version": "2012-10-17"
4747
},
4848
"ManagedPolicyArns": [
49-
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
49+
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
5050
],
5151
"Tags": [
5252
{
@@ -103,7 +103,7 @@
103103
"Version": "2012-10-17"
104104
},
105105
"ManagedPolicyArns": [
106-
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
106+
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
107107
],
108108
"Tags": [
109109
{
@@ -160,7 +160,7 @@
160160
"Version": "2012-10-17"
161161
},
162162
"ManagedPolicyArns": [
163-
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
163+
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
164164
],
165165
"Tags": [
166166
{

tests/translator/output/aws-cn/globals_for_function.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@
116116
"Version": "2012-10-17"
117117
},
118118
"ManagedPolicyArns": [
119-
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole",
119+
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy",
120120
"arn:aws-cn:iam::aws:policy/AWSXRayDaemonWriteAccess",
121121
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaVPCAccessExecutionRole"
122122
],
@@ -254,7 +254,7 @@
254254
"Version": "2012-10-17"
255255
},
256256
"ManagedPolicyArns": [
257-
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole",
257+
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy",
258258
"arn:aws-cn:iam::aws:policy/AWSXRayDaemonWriteAccess",
259259
"arn:aws-cn:iam::aws:policy/service-role/AWSLambdaVPCAccessExecutionRole"
260260
],

tests/translator/output/aws-us-gov/function_with_durable_config.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
"Version": "2012-10-17"
4747
},
4848
"ManagedPolicyArns": [
49-
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
49+
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
5050
],
5151
"Tags": [
5252
{

tests/translator/output/aws-us-gov/function_with_durable_config_globals.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
"Version": "2012-10-17"
4747
},
4848
"ManagedPolicyArns": [
49-
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
49+
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
5050
],
5151
"Tags": [
5252
{
@@ -103,7 +103,7 @@
103103
"Version": "2012-10-17"
104104
},
105105
"ManagedPolicyArns": [
106-
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
106+
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
107107
],
108108
"Tags": [
109109
{
@@ -160,7 +160,7 @@
160160
"Version": "2012-10-17"
161161
},
162162
"ManagedPolicyArns": [
163-
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
163+
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
164164
],
165165
"Tags": [
166166
{

tests/translator/output/aws-us-gov/globals_for_function.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@
116116
"Version": "2012-10-17"
117117
},
118118
"ManagedPolicyArns": [
119-
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole",
119+
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy",
120120
"arn:aws-us-gov:iam::aws:policy/AWSXRayDaemonWriteAccess",
121121
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaVPCAccessExecutionRole"
122122
],
@@ -254,7 +254,7 @@
254254
"Version": "2012-10-17"
255255
},
256256
"ManagedPolicyArns": [
257-
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole",
257+
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy",
258258
"arn:aws-us-gov:iam::aws:policy/AWSXRayDaemonWriteAccess",
259259
"arn:aws-us-gov:iam::aws:policy/service-role/AWSLambdaVPCAccessExecutionRole"
260260
],

tests/translator/output/function_with_durable_config.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
"Version": "2012-10-17"
4747
},
4848
"ManagedPolicyArns": [
49-
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
49+
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
5050
],
5151
"Tags": [
5252
{

tests/translator/output/function_with_durable_config_globals.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
"Version": "2012-10-17"
4747
},
4848
"ManagedPolicyArns": [
49-
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
49+
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
5050
],
5151
"Tags": [
5252
{
@@ -103,7 +103,7 @@
103103
"Version": "2012-10-17"
104104
},
105105
"ManagedPolicyArns": [
106-
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
106+
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
107107
],
108108
"Tags": [
109109
{
@@ -160,7 +160,7 @@
160160
"Version": "2012-10-17"
161161
},
162162
"ManagedPolicyArns": [
163-
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole"
163+
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy"
164164
],
165165
"Tags": [
166166
{

tests/translator/output/globals_for_function.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@
116116
"Version": "2012-10-17"
117117
},
118118
"ManagedPolicyArns": [
119-
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole",
119+
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy",
120120
"arn:aws:iam::aws:policy/AWSXrayWriteOnlyAccess",
121121
"arn:aws:iam::aws:policy/service-role/AWSLambdaVPCAccessExecutionRole"
122122
],
@@ -254,7 +254,7 @@
254254
"Version": "2012-10-17"
255255
},
256256
"ManagedPolicyArns": [
257-
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicExecutionRole",
257+
"arn:aws:iam::aws:policy/service-role/AWSLambdaBasicDurableExecutionRolePolicy",
258258
"arn:aws:iam::aws:policy/AWSXrayWriteOnlyAccess",
259259
"arn:aws:iam::aws:policy/service-role/AWSLambdaVPCAccessExecutionRole"
260260
],

0 commit comments

Comments
 (0)