Skip to content

Conversation

@akashveramd
Copy link

In this PR, added the condition in linux_job_v2.yml to run workflow called by ROCm job only on cron schedule and when push to main. This feature is optional and shouldn't affect other workflows. The calling ROCm workflow must set the input 'check-run-condition' to true for using this condition.
This effort is to limit the PR runs on ROCm runner due to limited CI resources and is part of torchtitan PR pytorch/torchtitan#2018

…b on cron schedule and when push to main only.
@vercel
Copy link

vercel bot commented Nov 12, 2025

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

A member of the Team first needs to authorize it.

@meta-cla meta-cla 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 Nov 12, 2025
@akashveramd
Copy link
Author

@huydhn @malfet: Can I please get a review for this PR?

Copy link
Member

@seemethere seemethere left a comment

Choose a reason for hiding this comment

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

No, I don't want to add a rocm specific conditional to the generic job

@akashveramd
Copy link
Author

No, I don't want to add a rocm specific conditional to the generic job

I see. However, I think we already have rocm specific conditionals https://github.com/pytorch/test-infra/blob/main/.github/workflows/linux_job_v2.yml#L169 in the existing file?

@seemethere
Copy link
Member

seemethere commented Nov 13, 2025

No, I don't want to add a rocm specific conditional to the generic job

I see. However, I think we already have rocm specific conditionals main/.github/workflows/linux_job_v2.yml#L169 in the existing file?

That conditional makes sense, the one you are introducing unfortunately does not.

If you want this conditional make it upstream in the torchtitan repository.

@akashveramd akashveramd reopened this Nov 13, 2025
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. module: rocm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants