Skip to content

Commit

Permalink
Restrict extra permissions for ECR private repo as Lambda team has co…
Browse files Browse the repository at this point in the history
…nfirmed they are unnecessary (#380)

Remove the extra ECR policy actions ecr:DeleteRepositoryPolicy, ecr:GetRepositoryPolicy, and ecr:SetRepositoryPolicy from our CloudFormation template.
  • Loading branch information
hehe7318 authored Jan 8, 2025
1 parent ac5d9c4 commit 1fc1a99
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions infrastructure/parallelcluster-ui.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -575,17 +575,6 @@ Resources:
Condition:
StringLike:
aws:SourceArn: !Sub arn:${AWS::Partition}:lambda:${AWS::Region}:${AWS::AccountId}:function:*
- Sid: ReadWriteEcrPolicy
Effect: Allow
Principal:
Service: !Sub lambda.${AWS::URLSuffix}
Action:
- ecr:DeleteRepositoryPolicy
- ecr:GetRepositoryPolicy
- ecr:SetRepositoryPolicy
Condition:
StringLike:
aws:SourceArn: !Sub arn:${AWS::Partition}:lambda:${AWS::Region}:${AWS::AccountId}:function:*

ImageBuilderInstanceRole:
Type: AWS::IAM::Role
Expand Down

0 comments on commit 1fc1a99

Please sign in to comment.