Skip to content

Conversation

@fduwjj
Copy link
Contributor

@fduwjj fduwjj commented Nov 22, 2022

Stack from ghstack (oldest at bottom):

Differential Revision: D41497546

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 22, 2022

🔗 Helpful Links

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

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

❌ 1 Failures

As of commit daaad36:

The following jobs have failed:

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

@fduwjj fduwjj added release notes: distributed (dtensor) release notes category and removed release notes: distributed (fsdp) release notes category labels Nov 22, 2022
@fduwjj
Copy link
Contributor Author

fduwjj commented Nov 23, 2022

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 23, 2022
@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: The following mandatory check(s) failed (Rule Distributed):

Dig deeper by viewing the failures on hud

Details for Dev Infra team Raised by workflow job

@fduwjj
Copy link
Contributor Author

fduwjj commented Nov 23, 2022

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a rebase job. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Rebase failed due to Command git -C /home/runner/work/pytorch/pytorch rebase refs/remotes/origin/viable/strict gh/fduwjj/49/orig returned non-zero exit code 1

Rebasing (1/1)
Auto-merging test/distributed/_tensor/parallel/test_parallelize_api.py
CONFLICT (content): Merge conflict in test/distributed/_tensor/parallel/test_parallelize_api.py
Auto-merging test/distributed/_tensor/parallel/test_tp_examples.py
Auto-merging test/distributed/_tensor/parallel/test_tp_style.py
Auto-merging torch/distributed/_tensor/parallel/__init__.py
CONFLICT (content): Merge conflict in torch/distributed/_tensor/parallel/__init__.py
Auto-merging torch/distributed/_tensor/parallel/api.py
CONFLICT (content): Merge conflict in torch/distributed/_tensor/parallel/api.py
Auto-merging torch/distributed/_tensor/parallel/style.py
Auto-merging torch/distributed/_tensor/parallel/utils.py
error: could not apply 802069ecb2... [PT-D][3/N] Sync TP API change to Pytorch
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply 802069ecb2... [PT-D][3/N] Sync TP API change to Pytorch

Raised by https://github.com/pytorch/pytorch/actions/runs/3529305373

fduwjj added a commit that referenced this pull request Nov 23, 2022
ghstack-source-id: 4409f99
Pull Request resolved: #89535
@fduwjj
Copy link
Contributor Author

fduwjj commented Nov 23, 2022

Manually rebase and addressed the comment.

@fduwjj
Copy link
Contributor Author

fduwjj commented Nov 23, 2022

@pytorchbot merge

@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: 2 additional jobs have failed, first few of them are: trunk ,trunk / cuda11.6-py3.10-gcc7-sm86 / test (default, 1, 4, linux.g5.4xlarge.nvidia.gpu)

Details for Dev Infra team Raised by workflow job

@fduwjj
Copy link
Contributor Author

fduwjj commented Nov 23, 2022

@pytorchbot -f merge "Trunk is broken not this PR"

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 23, 2022

❌ 🤖 pytorchbot command failed:

@pytorchbot: error: unrecognized arguments: -f Trunk is broken not this PR

usage: @pytorchbot [-h] {merge,revert,rebase,label,drci} ...

Try @pytorchbot --help for more info.

@fduwjj
Copy link
Contributor Author

fduwjj commented Nov 23, 2022

@pytorchbot merge -f "Trunk is broken not this PR"

@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).

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

@fduwjj
Copy link
Contributor Author

fduwjj commented Nov 23, 2022

@fduwjj has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
@facebook-github-bot facebook-github-bot deleted the gh/fduwjj/49/head branch June 8, 2023 17:11
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 release notes: distributed (dtensor) release notes category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants