Skip to content

Exclude unstable jobs when evaluating red % metrics#4246

Merged
huydhn merged 1 commit intopytorch:mainfrom
huydhn:ignore-unstable-jobs
Jun 2, 2023
Merged

Exclude unstable jobs when evaluating red % metrics#4246
huydhn merged 1 commit intopytorch:mainfrom
huydhn:ignore-unstable-jobs

Conversation

@huydhn
Copy link
Copy Markdown
Contributor

@huydhn huydhn commented Jun 1, 2023

After pytorch/pytorch#102426 and pytorch/pytorch#102784 landed, unstable jobs are now hidden correctly on HUD https://hud.pytorch.org and also won't block PR. Previously, this was done by moving unstable jobs to an unstable workflow. Now unstable jobs will stay in the same workflow, but have unstable in their names.

This is very similar to how rerun_disabled_tests are ignored atm.

Testing

https://torchci-git-fork-huydhn-ignore-unstable-jobs-fbopensource.vercel.app/metrics

@vercel
Copy link
Copy Markdown

vercel bot commented Jun 1, 2023

@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 Jun 1, 2023
@vercel
Copy link
Copy Markdown

vercel bot commented Jun 1, 2023

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

Name Status Preview Comments Updated (UTC)
torchci ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 1, 2023 10:07pm

@huydhn huydhn requested review from a team, ZainRizvi and clee2000 June 1, 2023 22:10
@huydhn huydhn marked this pull request as ready for review June 1, 2023 22:10
@huydhn huydhn merged commit bd2073d into pytorch:main Jun 2, 2023
PaliC pushed a commit that referenced this pull request Jun 5, 2023
After pytorch/pytorch#102426 and
pytorch/pytorch#102784 landed, unstable jobs are
now hidden correctly on HUD https://hud.pytorch.org and also won't block
PR. Previously, this was done by moving unstable jobs to an unstable
workflow. Now unstable jobs will stay in the same workflow, but have
`unstable` in their names.

This is very similar to how `rerun_disabled_tests` are ignored atm.

### Testing


https://torchci-git-fork-huydhn-ignore-unstable-jobs-fbopensource.vercel.app/metrics
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