Skip to content

change: add sa-west-1 to PyTorch training image URI config - #6516

Merged
mufaddal-rohawala merged 4 commits into
aws:masterfrom
Yadan-Wei:pytorch-sa-west-1
Oct 23, 2026
Merged

mufaddal-rohawala merged 4 commits into
aws:masterfrom
Yadan-Wei:pytorch-sa-west-1

Conversation

@Yadan-Wei

Copy link
Copy Markdown
Contributor

Issue #, if available:

Description of changes:

Adds the sa-west-1 registry (434457397001) to the PyTorch training image URI config for versions 2.8.0, 2.9.0 and 2.10.0. The CPU and GPU SageMaker images for these versions are available in that region.

This is the pytorch.json counterpart of #6507, which adds sa-west-1 to the other image URI configs. Only these three registry entries change; the rest of the file is untouched.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Yadan-Wei
Yadan-Wei deployed to auto-approve October 21, 2026 16:55 — with GitHub Actions Active
@Yadan-Wei
Yadan-Wei deployed to auto-approve October 21, 2026 16:57 — with GitHub Actions Active
@Yadan-Wei
Yadan-Wei deployed to auto-approve October 21, 2026 16:57 — 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 the sa-west-1 registry entry (434457397001) to the PyTorch training image URI config for versions 2.8.0, 2.9.0, and 2.10.0. The additions are data-only, alphabetically placed between sa-east-1 and us-east-1, and use the same sa-west-1 account ID established in PR #6478. The change is backward compatible and matches the PR's stated scope. The PR looks clean; no issues found.

Reviewed commit 70718e6. Automated review; verify before acting.

@Yadan-Wei
Yadan-Wei deployed to auto-approve October 22, 2026 19:06 — with GitHub Actions Active
@Yadan-Wei
Yadan-Wei deployed to auto-approve October 22, 2026 19:06 — with GitHub Actions Active
@Yadan-Wei
Yadan-Wei deployed to auto-approve October 22, 2026 19:07 — 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 the sa-west-1 registry (434457397001) to the PyTorch training image URI config for versions 2.8.0, 2.9.0, and 2.10.0. The entries are inserted in correct alphabetical order (between sa-east-1 and us-east-1), the account ID matches the sa-west-1 registry account established in precedent #6478, and the change is a backward-compatible data-only addition. No issues found.

Reviewed commit 52ce79d. Automated review; verify before acting.

@Yadan-Wei
Yadan-Wei deployed to auto-approve October 22, 2026 23:40 — with GitHub Actions Active
@Yadan-Wei
Yadan-Wei deployed to auto-approve October 22, 2026 23:40 — with GitHub Actions Active
@Yadan-Wei
Yadan-Wei deployed to auto-approve October 22, 2026 23:40 — 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 the sa-west-1 registry (434457397001) to the PyTorch training image URI config for versions 2.8.0, 2.9.0 and 2.10.0 in pytorch.json. The change is data-only: the account ID matches the sa-west-1 precedent from PR #6478, entries are placed in correct alphabetical order (after sa-east-1), and all three versions are updated consistently within the training scope. No code issues found. As a non-blocking note, per prior guidance (Source: #6478), please confirm the PyTorch CPU/GPU images are actually published in sa-west-1 upstream before merge.

Reviewed commit 4b537f0. Automated review; verify before acting.

@Yadan-Wei
Yadan-Wei deployed to auto-approve October 23, 2026 17:52 — with GitHub Actions Active
@Yadan-Wei
Yadan-Wei deployed to auto-approve October 23, 2026 17:52 — with GitHub Actions Active
@Yadan-Wei
Yadan-Wei deployed to auto-approve October 23, 2026 17:52 — 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 is a data-only change adding the sa-west-1 registry (434457397001) to the PyTorch training image URI config for versions 2.8.0, 2.9.0 and 2.10.0. The account ID matches the established sa-west-1 precedent (PR #6478), the entries are inserted in correct alphabetical order between sa-east-1 and us-east-1, and no version aliases or existing region sets are narrowed, so there is no backward-compatibility risk. The change looks clean.

Reviewed commit c8e17ca. Automated review; verify before acting.

@mufaddal-rohawala
mufaddal-rohawala merged commit bdb9eee into aws:master Oct 23, 2026
25 of 30 checks passed
@Yadan-Wei
Yadan-Wei deleted the pytorch-sa-west-1 branch October 23, 2026 22:34

This branch was successfully deployed

1 active deployment
auto-approve — c8e17ca2 Deployed Oct 23, 2026 by Yadan-Wei via wait-for-approval #1830
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