Skip to content

Commit 1006ee3

Browse files
author
Jaipreet Singh
committed
Fix resolution for role names with paths
1 parent b0b8d33 commit 1006ee3

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

sagemaker_studio_image_build/cli.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,11 +37,12 @@ def get_role(args):
3737
import sagemaker
3838

3939
logging.basicConfig(level=logging.INFO)
40-
41-
return sagemaker.get_execution_role().split("/")[-1]
40+
# arn:aws:iam::$account_id:role/$path/$name -> $path/$name
41+
return "/".join(sagemaker.get_execution_role().split(":")[-1].split("/")[1:])
4242
except ValueError as e:
4343
raise ValueError(
44-
"Unable to determine execution role. Please provide via --role", e
44+
"Unable to determine execution role. Please provide via the --role argument",
45+
e,
4546
)
4647

4748

setup.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
setuptools.setup(
99
name="sagemaker_studio_image_build",
10-
version="0.3",
10+
version="0.3.1",
1111
author="Amazon Web Services",
1212
description="Build Docker Images in Amazon SageMaker Studio using AWS CodeBuild",
1313
long_description=long_description,

0 commit comments

Comments
 (0)