Skip to content

Commit 468fc08

Browse files
committed
chore: log info msg when overriding jumpstart bucket
1 parent af5e1d4 commit 468fc08

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed

src/sagemaker/jumpstart/utils.py

+3-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,9 @@ def get_jumpstart_content_bucket(region: str) -> str:
6666
constants.ENV_VARIABLE_JUMPSTART_CONTENT_BUCKET_OVERRIDE in os.environ
6767
and len(os.environ[constants.ENV_VARIABLE_JUMPSTART_CONTENT_BUCKET_OVERRIDE]) > 0
6868
):
69-
return os.environ[constants.ENV_VARIABLE_JUMPSTART_CONTENT_BUCKET_OVERRIDE]
69+
bucket_override = os.environ[constants.ENV_VARIABLE_JUMPSTART_CONTENT_BUCKET_OVERRIDE]
70+
LOGGER.info("Using JumpStart bucket override: '%s'", bucket_override)
71+
return bucket_override
7072
try:
7173
return constants.JUMPSTART_REGION_NAME_TO_LAUNCHED_REGION_DICT[region].content_bucket
7274
except KeyError:

tests/unit/sagemaker/jumpstart/test_utils.py

+7-2
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,13 @@ def test_get_jumpstart_content_bucket():
4444

4545
def test_get_jumpstart_content_bucket_override():
4646
with patch.dict(os.environ, {ENV_VARIABLE_JUMPSTART_CONTENT_BUCKET_OVERRIDE: "some-val"}):
47-
random_region = "random_region"
48-
assert "some-val" == utils.get_jumpstart_content_bucket(random_region)
47+
with patch("logging.Logger.info") as mocked_info_log:
48+
random_region = "random_region"
49+
assert "some-val" == utils.get_jumpstart_content_bucket(random_region)
50+
mocked_info_log.assert_called_once_with(
51+
"Using JumpStart bucket override: '%s'",
52+
"some-val",
53+
)
4954

5055

5156
def test_get_jumpstart_launched_regions_message():

0 commit comments

Comments
 (0)