Skip to content

Conversation

@huydhn
Copy link
Contributor

@huydhn huydhn commented Nov 12, 2024

This is to prevent regression like #6564. If nightly is accidentally used on CI instead of PT built from the pinned commit at ci_commit_pins/pytorch.txt. The check will fail.

Because setup-linux script is used on both ET and PT CI, the latter will also have this check in place after the next ET pin commit update.

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 12, 2024

🔗 Helpful Links

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

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

❌ 1 New Failure

As of commit befe2d2 with merge base c8967da (image):

NEW FAILURE - The following job has failed:

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

@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 Nov 12, 2024
@huydhn huydhn marked this pull request as ready for review November 12, 2024 18:43
@huydhn huydhn merged commit f8fda0a into main Nov 12, 2024
64 of 65 checks passed
@huydhn huydhn deleted the fail-ci-when-using-nightly branch November 12, 2024 20:18
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