Skip to content

Commit cd1658a

Browse files
committed
Fix region issue for studio url
1 parent 928204b commit cd1658a

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

sagemaker-train/src/sagemaker/train/common_utils/metrics_visualizer.py

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -94,11 +94,10 @@ def get_studio_url(training_job, domain_id: str = None) -> str:
9494
Example:
9595
>>> from sagemaker.train import get_studio_url
9696
>>> url = get_studio_url('my-training-job')
97-
>>> url = get_studio_url('arn:aws:sagemaker:us-east-1:123456789:training-job/my-job')
97+
>>> url = get_studio_url('arn:aws:sagemaker:us-west-2:123456789:training-job/my-job')
9898
"""
9999
import re
100100

101-
# Handle ARN string — extract region and job name directly
102101
if isinstance(training_job, str):
103102
arn_match = re.match(
104103
r'arn:aws(?:-[a-z]+)?:sagemaker:([a-z0-9-]+):\d+:training-job/(.+)',
@@ -108,12 +107,14 @@ def get_studio_url(training_job, domain_id: str = None) -> str:
108107
region = arn_match.group(1)
109108
job_name = arn_match.group(2)
110109
else:
111-
# Treat as job name, need to fetch the object
110+
# Plain job name — use session region
112111
training_job = TrainingJob.get(training_job_name=training_job)
113-
region = training_job.region if hasattr(training_job, 'region') and training_job.region else 'us-east-1'
112+
from sagemaker.core.utils.utils import SageMakerClient
113+
region = SageMakerClient().region_name
114114
job_name = training_job.training_job_name
115115
else:
116-
region = training_job.region if hasattr(training_job, 'region') and training_job.region else 'us-east-1'
116+
from sagemaker.core.utils.utils import SageMakerClient
117+
region = SageMakerClient().region_name
117118
job_name = training_job.training_job_name
118119

119120
base = _get_studio_base_url(region)

0 commit comments

Comments
 (0)