Skip to content

Fix stale Dr. CI comment after retrying jobs#851

Merged
huydhn merged 2 commits intopytorch:mainfrom
huydhn:fix-stale-dr-ci-comment
Oct 6, 2022
Merged

Fix stale Dr. CI comment after retrying jobs#851
huydhn merged 2 commits intopytorch:mainfrom
huydhn:fix-stale-dr-ci-comment

Conversation

@huydhn
Copy link
Contributor

@huydhn huydhn commented Oct 6, 2022

Recently, I realize that Dr. CI doesn't take the results of retrying jobs into account and this leads to a discrepancy between Dr. CI comment, HUD, and GitHub signal box. This is an example pytorch/pytorch#85313 where Dr. CI reports a failing job while none is showed in https://hud.pytorch.org/pr/85313 or GitHub signal box. The failed jobs was retried successfully.

Also cleaning up the recent_pr_workflows_query query a bit and adding run_attempt field.

@huydhn huydhn requested review from a team and clee2000 October 6, 2022 05:12
@vercel
Copy link

vercel bot commented Oct 6, 2022

@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 Oct 6, 2022
@vercel
Copy link

vercel bot commented Oct 6, 2022

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

Name Status Preview Updated
torchci ✅ Ready (Inspect) Visit Preview Oct 6, 2022 at 5:16AM (UTC)

@huydhn huydhn marked this pull request as ready for review October 6, 2022 05:16
@huydhn huydhn requested a review from ZainRizvi October 6, 2022 17:41
Copy link
Contributor

@ZainRizvi ZainRizvi left a comment

Choose a reason for hiding this comment

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

🚀🚀🚀

@huydhn huydhn merged commit f0f6b0e into pytorch:main Oct 6, 2022
@huydhn huydhn deleted the fix-stale-dr-ci-comment branch October 7, 2022 00:22
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