Skip to content

Commit

Permalink
remove datalake admin role
Browse files Browse the repository at this point in the history
only used by the replicate lambda
  • Loading branch information
cnfait committed Oct 1, 2024
1 parent 84865dc commit e470864
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions sdlf-foundations/src/template.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,7 @@ Resources:
# Role defined upstream due to Lake Formation PutDataLakeSettings constraints
rDataLakeAdminRole:
Type: AWS::IAM::Role
Condition: UseLegacyTables
Properties:
AssumeRolePolicyDocument:
Version: 2012-10-17
Expand Down Expand Up @@ -158,9 +159,11 @@ Resources:
rDataLakeSettings:
Type: AWS::LakeFormation::DataLakeSettings
Properties:
Admins:
- DataLakePrincipalIdentifier: !GetAtt rDataLakeAdminRole.Arn
- DataLakePrincipalIdentifier: !Sub arn:${AWS::Partition}:iam::${AWS::AccountId}:role/${pCicdRole}
Admins: !If
- UseLegacyTables
- - DataLakePrincipalIdentifier: !GetAtt rDataLakeAdminRole.Arn
- DataLakePrincipalIdentifier: !Sub arn:${AWS::Partition}:iam::${AWS::AccountId}:role/${pCicdRole}
- - DataLakePrincipalIdentifier: !Sub arn:${AWS::Partition}:iam::${AWS::AccountId}:role/${pCicdRole}
CreateDatabaseDefaultPermissions: []
CreateTableDefaultPermissions: []
MutationType: REPLACE
Expand Down Expand Up @@ -1563,6 +1566,7 @@ Resources:

rDataLakeAdminRoleSsm:
Type: AWS::SSM::Parameter
Condition: UseLegacyTables
Properties:
Name: /SDLF/IAM/DataLakeAdminRoleArn
Type: String
Expand Down

0 comments on commit e470864

Please sign in to comment.