KEMBAR78
Give the magma build job id-token write permissions by afrittoli · Pull Request #140141 · pytorch/pytorch · GitHub
Skip to content

Conversation

@afrittoli
Copy link
Collaborator

The configure-aws-credentials action requires special permissions: https://github.com/aws-actions/configure-aws-credentials?tab=readme-ov-file#oidc

Give "id-token: write" permssion to the job that sets the AWS credentials to upload to the S3 bucket.

Fixes #139397

The configure-aws-credentials action requires special permissions:
https://github.com/aws-actions/configure-aws-credentials?tab=readme-ov-file#oidc

Give "id-token: write" permssion to the job that sets the AWS
credentials to upload to the S3 bucket.

Signed-off-by: Andrea Frittoli <andrea.frittoli@uk.ibm.com>
@afrittoli afrittoli requested a review from a team as a code owner November 8, 2024 14:48
@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Nov 8, 2024
@pytorch-bot
Copy link

pytorch-bot bot commented Nov 8, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/140141

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 012ab6a with merge base c1c94cb (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

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

lgtm

@atalman
Copy link
Contributor

atalman commented Nov 8, 2024

@pytorchmergebot merge -f "lint is green"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

Ryo-not-rio pushed a commit to Ryo-not-rio/pytorch that referenced this pull request Dec 2, 2024
The configure-aws-credentials action requires special permissions: https://github.com/aws-actions/configure-aws-credentials?tab=readme-ov-file#oidc

Give "id-token: write" permssion to the job that sets the AWS credentials to upload to the S3 bucket.

Fixes pytorch#139397

Pull Request resolved: pytorch#140141
Approved by: https://github.com/atalman
pobin6 pushed a commit to pobin6/pytorch that referenced this pull request Dec 5, 2024
The configure-aws-credentials action requires special permissions: https://github.com/aws-actions/configure-aws-credentials?tab=readme-ov-file#oidc

Give "id-token: write" permssion to the job that sets the AWS credentials to upload to the S3 bucket.

Fixes pytorch#139397

Pull Request resolved: pytorch#140141
Approved by: https://github.com/atalman
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build a non-conda package for magma-cuda

4 participants