Skip to content

Conversation

@huydhn
Copy link
Contributor

@huydhn huydhn commented Apr 25, 2024

This is the next part of #5122 where Dr.CI also uses the list of unstable issues to check if a job is an unstable job. The same function isUnstableJob that powers HUD from #5122 is used.

Testing

Mark a job on ET as unstable pytorch/executorch#3344 as shown on HUD https://hud.pytorch.org/hud/pytorch/executorch/main and Dr.CI applies the same logic on a PR pytorch/executorch#3318 to show the job as unstable.

curl --request POST \
--url "http://localhost:3000/api/drci/drci?prNumber=3318" \
--header "Authorization: TOKEN" \
--data 'repo=executorch'

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/3318

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (1 Unrelated Failure)

As of commit f712e381c161901b733baa6b1fe7d85dc25404d3 with merge base b669056c1cff5f7fe3786df9e68a14447cd5410b (image):

UNSTABLE - The following job failed but was likely due to flakiness present on trunk and has been marked as unstable:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@huydhn huydhn requested a review from clee2000 April 25, 2024 02:50
@vercel
Copy link

vercel bot commented Apr 25, 2024

@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 Apr 25, 2024
@vercel
Copy link

vercel bot commented Apr 25, 2024

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 Apr 26, 2024 0:51am

@huydhn huydhn merged commit 673d728 into pytorch:main Apr 26, 2024
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Apr 26, 2024
…24965)

After pytorch/test-infra#5131, we want to have trymerge to honor the list of unstable failures from Dr.CI because having the unstable keyword is the job name now doesn't cover all unstable jobs.
Pull Request resolved: #124965
Approved by: https://github.com/clee2000
petrex pushed a commit to petrex/pytorch that referenced this pull request May 3, 2024
…torch#124965)

After pytorch/test-infra#5131, we want to have trymerge to honor the list of unstable failures from Dr.CI because having the unstable keyword is the job name now doesn't cover all unstable jobs.
Pull Request resolved: pytorch#124965
Approved by: https://github.com/clee2000
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