Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ability to enable/disable linux_job_v2 workflow calls #6309

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jithunnair-amd
Copy link
Collaborator

@jithunnair-amd jithunnair-amd commented Feb 20, 2025

Related to pytorch/ao#1581

Helps to enable/disable specific entries of a matrix that are all invoking linux_job_v2.yml. In the case of the torchao PR above, we want to disable only the ROCm entry of the matrix when the workflow runs on torchao PRs.

Example of successful disable using this PR logic: https://github.com/pytorch/ao/actions/runs/13444026734/workflow

image

Copy link

vercel bot commented Feb 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
torchci ⬜️ Ignored (Inspect) Visit Preview Feb 20, 2025 8:27pm

@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 Feb 20, 2025
@jithunnair-amd jithunnair-amd marked this pull request as ready for review February 20, 2025 20:34
@jithunnair-amd
Copy link
Collaborator Author

@atalman The only caveat is that it seems Github Actions doesn't process all the inputs when it skips the ROCm workflow call, so it shows ${{ inputs.job-name }} on the UI, instead of linux-job. I'm not sure if there is a way to resolve this.
cc @huydhn

@huydhn
Copy link
Contributor

huydhn commented Feb 20, 2025

I have a comment on https://github.com/pytorch/ao/pull/1581/files#r1964465401 to see if this is really needed. This doesn't seem like a good idea to have this custom flag here to sever just this case

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