Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Generalized AWS Principal statement to contain generic info.

Co-authored-by: Gabriele Fedi <[email protected]>
  • Loading branch information
gvasquezvargas and GabriFedi97 committed Jun 26, 2024
1 parent 8ea7bcf commit abd7e50
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,7 @@ After you create the cluster in the BigAnimal console, the UI will display the *
"Sid": "Enable IAM User Permissions",
"Effect": "Allow",
"Principal": {
"AWS": "arn:aws:iam::142348057180:root"
"AWS": "arn:aws:iam::<aws_project_id>:root"
},
"Action": "kms:*",
"Resource": "*"
Expand All @@ -262,7 +262,7 @@ After you create the cluster in the BigAnimal console, the UI will display the *
"Sid": "Enable TDE on cluster ExampleCluster",
"Effect": "Allow",
"Principal": {
"AWS": "arn:aws:iam::142348057180:role/ba-vcvolsnap0ZUPsgZ-us-east-1-p-k2nxxtndkg"
"AWS": "arn:aws:iam::<aws_project_id>:role/<pg_cluster_role>"
},
"Action": [
"kms:Encrypt",
Expand Down

0 comments on commit abd7e50

Please sign in to comment.