Skip to content

Conversation

@huydhn
Copy link
Contributor

@huydhn huydhn commented Sep 24, 2024

As reported by @guangy10 in
pytorch/executorch#5441 (comment), it turns out that the job timeout can be set to a higher value (120 minutes), but the AWS credential still expires after 1 hour https://github.com/aws-actions/configure-aws-credentials.

The fix here is to set the role duration correctly. However, the max that it can go is 18000 seconds. I believe that's plenty of room already, but if a higher is needed, we will need to update the server side
https://github.com/pytorch-labs/pytorch-gha-infra/blob/main/runners/gha_roles.tf#L1425

Testing

The role duration is set correctly to 7200 seconds (2 hours) https://github.com/pytorch/test-infra/actions/runs/10916800466/job/30298898748#step:4:4

As reported by @guangy10 in
pytorch/executorch#5441 (comment),
it turns out that the job timeout can be set to a higher value (120
minutes), but the AWS credential still expires after 1 hour
https://github.com/aws-actions/configure-aws-credentials.

The fix here is to set the role duration correctly. However, the max
that it can go is 18000 seconds. I believe that's plenty of room
already, but if a higher is needed, we will need to update the server
side
https://github.com/pytorch-labs/pytorch-gha-infra/blob/main/runners/gha_roles.tf#L1425

### Testing

The role duration is set correctly to 7200 seconds (2 hours)
https://github.com/pytorch/test-infra/actions/runs/10916800466/job/30298898748#step:4:4
@huydhn huydhn requested review from atalman and kit1980 September 24, 2024 22:11
@vercel
Copy link

vercel bot commented Sep 24, 2024

@huydhn is attempting to deploy a commit to the Meta Open Source Team on Vercel.

A member of the Team first needs to authorize it.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 24, 2024
@kit1980 kit1980 merged commit 170364c into pytorch:release/2.5 Sep 24, 2024
3 of 6 checks passed
@huydhn huydhn deleted the cherry-pick-5685 branch December 27, 2024 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants