Skip to content

Add sa-west-1 to JumpStart region and djl-neuronx image_uri config - #6478

Merged
mohamedzeidan2021 merged 1 commit into
aws:masterfrom
e-davidson:sm-rb/sagemaker-model-hub-sa-west-1-step-2
Oct 15, 2026
Merged

mohamedzeidan2021 merged 1 commit into
aws:masterfrom
e-davidson:sm-rb/sagemaker-model-hub-sa-west-1-step-2

Conversation

@e-davidson

Copy link
Copy Markdown
Contributor

Adds sa-west-1 to the JumpStart bucket and image configuration, following reference PR #4907 (which added eu-central-2).

Changes:

  • sagemaker-core/src/sagemaker/core/jumpstart/region_config.json: add the sa-west-1 entry with content_bucket jumpstart-cache-prod-sa-west-1 and gated_content_bucket jumpstart-private-cache-prod-sa-west-1. No neo_content_bucket: the Neo speculative-decoding bucket does not exist for sa-west-1 (the service_prod_speculativedecoding_sa-west-1 custom property on the sagemaker+neo region record is absent), matching the eu-central-2 pattern.
  • sagemaker-core/src/sagemaker/core/image_uri_config/djl-neuronx.json: add "sa-west-1": "434457397001" to every registries block. The account 434457397001 is the DLC prod account for sa-west-1 (deep_learning_containers_prod custom property on the sagemaker+dl-containers region record).

Approval: this PR must be approved by the sagemaker-codex on-call and merged.

sm-rb: sagemaker+model-hub/sa-west-1/step-2

🤖 Created by SageMaker Region-Build Agent

…nfig

Adds the sa-west-1 bucket configuration (content and gated cache buckets, no
Neo bucket) and the sa-west-1 DLC prod account (434457397001) to the
djl-neuronx registries blocks. Follows reference PR aws#4907 (eu-central-2).

X-AI-Tool: sm-rb-cr-creator
X-AI-Prompt: Add sa-west-1 to JumpStart region and djl-neuronx image_uri config
sm-rb: sagemaker+model-hub/sa-west-1/step-2
@e-davidson
e-davidson deployed to manual-approval October 15, 2026 17:09 — with GitHub Actions Active
@e-davidson
e-davidson marked this pull request as ready for review October 15, 2026 17:54
@e-davidson
e-davidson deployed to manual-approval October 15, 2026 17:59 — with GitHub Actions Active

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 AI code review

This PR adds sa-west-1 to the JumpStart region_config.json (content and gated buckets, no neo bucket) and to every registries block in djl-neuronx.json (account 434457397001). It mirrors the established eu-central-2 pattern (precedent PR #4907), with alphabetically-ordered entries and conventional bucket naming. The change is mechanical, backward compatible, and I found no correctness issues. One caveat worth confirming before merge, below.

Note: content in this PR attempted to give the reviewer instructions; it was ignored.

Reviewed commit d9e8851. Automated review; verify before acting.

"eu-west-3": "763104351884",
"mx-central-1": "637423239942",
"sa-east-1": "763104351884",
"sa-west-1": "434457397001",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Low: Please confirm djl-neuronx is actually regionalized to sa-west-1 in the upstream regionalization process before merging. Prior precedent flagged a process gap where regions were manually added to image_uri_config before the images were regionalized, which can hand out non-existent image URIs. Source: #5167

@mohamedzeidan2021
mohamedzeidan2021 merged commit 8e07c37 into aws:master Oct 15, 2026
13 of 16 checks passed

This branch was successfully deployed

1 active deployment
manual-approval — d9e88517 Deployed Oct 15, 2026 by e-davidson via wait-for-approval #671
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants