KEMBAR78
[BE] Add script to keept the runner-determinator scripts in sync by ZainRizvi · Pull Request #136794 · pytorch/pytorch · GitHub
Skip to content

Conversation

@ZainRizvi
Copy link
Contributor

Whenever we update runner_determinator.py it needs to be copied over into _runner-determinator.yml.

This is a quick utility script to make that process less tedious

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Sep 26, 2024
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 26, 2024

🔗 Helpful Links

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

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

❌ 1 New Failure

As of commit 79159de with merge base 156ca01 (image):

NEW FAILURE - The following job has failed:

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

@ZainRizvi ZainRizvi marked this pull request as ready for review September 26, 2024 21:36
@ZainRizvi ZainRizvi requested a review from a team as a code owner September 26, 2024 21:36
Copy link
Collaborator

@zxiiro zxiiro left a comment

Choose a reason for hiding this comment

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

Nice!

Edit: Looks like we need to fix a lint issue.

@jeanschmidt
Copy link
Contributor

NEAT, but we need to get the lint right :)

@ZainRizvi
Copy link
Contributor Author

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Sep 30, 2024
@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: 1 jobs have failed, first few of them are: trunk / linux-focal-rocm6.2-py3.10 / test (default, 1, 2, linux.rocm.gpu)

Details for Dev Infra team Raised by workflow job

@ZainRizvi
Copy link
Contributor Author

@pytorchmergebot merge -i

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged while ignoring the following 1 checks: trunk / linux-focal-rocm6.2-py3.10 / test (default, 1, 2, linux.rocm.gpu)

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

AnantGulati pushed a commit to AnantGulati/pytorch that referenced this pull request Oct 2, 2024
…orch#136794)

Whenever we update runner_determinator.py it needs to be copied over into _runner-determinator.yml.

This is a quick utility script to make that process less tedious
Pull Request resolved: pytorch#136794
Approved by: https://github.com/zxiiro, https://github.com/jeanschmidt
@github-actions github-actions bot deleted the zainr/rd-updater branch November 3, 2024 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk Trigger trunk jobs on your pull request Merged topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants