Skip to content

test: unskip gpu intensive tests#5892

Open
lucasjia-aws wants to merge 2 commits into
aws:masterfrom
lucasjia-aws:unskip-gpu-intensive-tests
Open

test: unskip gpu intensive tests#5892
lucasjia-aws wants to merge 2 commits into
aws:masterfrom
lucasjia-aws:unskip-gpu-intensive-tests

Conversation

@lucasjia-aws
Copy link
Copy Markdown
Collaborator

@lucasjia-aws lucasjia-aws commented May 26, 2026

Continue developing on top of #5835

uses: aws-actions/configure-aws-credentials@v4
with:
role-to-assume: ${{ secrets.CI_AWS_ROLE_ARN }}
aws-region: us-west-2
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

region us-east-1?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

From my comprehension: the CodeBuild execution role possesses cross-region SageMaker permissions; the aws-region: us-west-2 setting in the GitHub Action only affects the STS endpoint used to trigger CodeBuild, and does not influence the region in which the tests are executed. So should be fine.

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