Skip to content

Conversation

@jithunnair-amd
Copy link
Contributor

@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

@vercel
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
Contributor 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

@jithunnair-amd
Copy link
Contributor Author

Closing as not required

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.

4 participants