Repository navigation
Add sa-west-1 to JumpStart region and djl-neuronx image_uri config - #6478
mohamedzeidan2021 merged 1 commit into
Conversation
…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
There was a problem hiding this comment.
🤖 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", |
There was a problem hiding this comment.
🟡 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
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 thesa-west-1entry withcontent_bucketjumpstart-cache-prod-sa-west-1andgated_content_bucketjumpstart-private-cache-prod-sa-west-1. Noneo_content_bucket: the Neo speculative-decoding bucket does not exist for sa-west-1 (theservice_prod_speculativedecoding_sa-west-1custom property on thesagemaker+neoregion 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 everyregistriesblock. The account434457397001is the DLC prod account for sa-west-1 (deep_learning_containers_prodcustom property on thesagemaker+dl-containersregion 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