[CI] Fix update slow tests #135390
[CI] Fix update slow tests #135390
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/135390
Note: Links to docs will display an error until the docs builds have been completed. This comment was automatically generated by Dr. CI and updates every 15 minutes. |
| mandatory_checks_name: | ||
| - EasyCLA | ||
| - Lint | ||
| - pull |
There was a problem hiding this comment.
Any idea why trunk isn't part of most of these rules?
There was a problem hiding this comment.
I think trunk used to mean only jobs running in trunk, not PR. Then we make it running as part of merge process, so it's not here.
| # no existing pr, so make a new one and approve it | ||
| pr_num = make_pr("pytorch/pytorch", params) | ||
| time.sleep(5) | ||
| add_labels("pytorch/pytorch", pr_num, ["ciflow/slow", "ci-no-td"]) |
There was a problem hiding this comment.
Alternatively, you could update titleRegexToLabel in https://github.com/pytorch/test-infra/blob/main/torchci/lib/bot/autoLabelBot.ts#L19 to add these labels. I think that would be cleaner than writing another add_labels function here
There was a problem hiding this comment.
I think I'd prefer to leave it here in order to keep everything in one place
| mandatory_checks_name: | ||
| - EasyCLA | ||
| - Lint | ||
| - pull |
There was a problem hiding this comment.
Any idea why trunk isn't part of most of these rules?
|
@pytorchbot merge -f "no trunk needed" |
Merge startedYour change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use Learn more about merging in the wiki. Questions? Feedback? Please reach out to the PyTorch DevX Team |
* Add pytorchbot to list of approvers for file * Add labels to the auto created PR The auto generated PR is currently not merging due to some failing tests on slow workflow that were supposed to be moved back to normal idk if this has much value, clearly we've been managing without the update Pull Request resolved: #135390 Approved by: https://github.com/ZainRizvi
* Add pytorchbot to list of approvers for file * Add labels to the auto created PR The auto generated PR is currently not merging due to some failing tests on slow workflow that were supposed to be moved back to normal idk if this has much value, clearly we've been managing without the update Pull Request resolved: pytorch#135390 Approved by: https://github.com/ZainRizvi
4 participants
The auto generated PR is currently not merging due to some failing tests on slow workflow that were supposed to be moved back to normal
idk if this has much value, clearly we've been managing without the update