Skip to content

Commit

Permalink
Merge pull request #184 from leoroy/fix-sp
Browse files Browse the repository at this point in the history
fix: fix the issue allowing unused service principals
  • Loading branch information
leoroy authored Mar 7, 2023
2 parents 8d4e0f6 + e482422 commit 42be48b
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions tests/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -319,9 +319,7 @@ def training_role_arn(boto3_session):
"Effect": "Allow",
"Principal": {
"Service": [
"sagemaker.amazonaws.com",
"maeve.aws.internal",
"ease.aws.internal"
"sagemaker.amazonaws.com"
]
},
"Action": "sts:AssumeRole"
Expand Down

0 comments on commit 42be48b

Please sign in to comment.