Skip to content

[testing] Add test owner labels for some distributed tests #163174

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[testing] Add test owner labels for some distributed tests #163174

wants to merge 1 commit into from

Conversation

clee2000
Copy link
Contributor

@clee2000 clee2000 commented Sep 17, 2025 •
edited
Loading

I am trying to give some test files better owner labels than module: unknown. I am not sure them, but they seem pretty reasonable

cc @H-Huang @awgu @wanchaol @fegin @fduwjj @wz337 @wconstab @d4l3k @pragupta @ezyang @msaroufim @dcci

@pytorch-bot PyTorch Bot
Copy link

pytorch-bot bot commented Sep 17, 2025 •
edited
Loading

🔗 Helpful Links

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

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

❌ 4 Cancelled Jobs

As of commit 86a86e8 with merge base e63476b (image):

CANCELLED JOBS - The following jobs were cancelled. Please retry:

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

@pytorch-bot pytorch-bot bot added ciflow/h100-symm-mem oncall: distributed Add this issue/PR to distributed oncall triage queue topic: not user facing topic category labels Sep 17, 2025
@clee2000 clee2000 marked this pull request as ready for review September 17, 2025 18:30
@clee2000 clee2000 requested review from a team and awgu September 17, 2025 18:31
@clee2000 clee2000 changed the title [testing] Give some distributed tests better owners than module unknown [testing] Add test owner labels for some distributed tests Sep 17, 2025
@ezyang
Copy link
Contributor

ezyang commented Sep 21, 2025

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Sep 21, 2025
@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: 3 mandatory check(s) failed. The first few are:

Dig deeper by viewing the failures on hud

Details for Dev Infra team Raised by workflow job

Failing merge rule: Core Maintainers

@clee2000
Copy link
Contributor Author

@pytorchbot merge -f "only care about lint"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

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

jainapurva pushed a commit that referenced this pull request Sep 29, 2025
@clee2000 @jainapurva
[testing] Add test owner labels for some distributed tests (#163174)
0c247ee 
I am trying to give some test files better owner labels than `module: unknown`.  I am not sure them, but they seem pretty reasonable

Pull Request resolved: #163174
Approved by: https://github.com/ezyang
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ezyang ezyang ezyang approved these changes

@awgu awgu Awaiting requested review from awgu

Assignees

No one assigned

Labels

ciflow/h100-symm-mem ciflow/trunk Trigger trunk jobs on your pull request Merged oncall: distributed Add this issue/PR to distributed oncall triage queue topic: not user facing topic category

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@clee2000 @ezyang @pytorchmergebot