Skip to content

[BE] Upload triage logs to S3#176320

Closed
malfet wants to merge 5 commits intogh/malfet/752/basefrom
gh/malfet/752/head
Closed

[BE] Upload triage logs to S3#176320
malfet wants to merge 5 commits intogh/malfet/752/basefrom
gh/malfet/752/head

Conversation

@malfet
Copy link
Contributor

@malfet malfet commented Mar 3, 2026

Stack from ghstack (oldest at bottom):

Co-authored by Claude

[ghstack-poisoned]
@malfet malfet requested a review from a team as a code owner March 3, 2026 18:28
@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Mar 3, 2026
@pytorch-bot
Copy link

pytorch-bot bot commented Mar 3, 2026

🔗 Helpful Links

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

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

⏳ No Failures, 27 Pending

As of commit faca4d8 with merge base a6461e6 (image):
💚 Looks good so far! There are no failures yet. 💚

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

malfet added a commit that referenced this pull request Mar 3, 2026
ghstack-source-id: 5671423
Pull-Request: #176320
[ghstack-poisoned]
malfet added a commit that referenced this pull request Mar 3, 2026
ghstack-source-id: 88e08ec
Pull-Request: #176320
[ghstack-poisoned]
malfet added a commit that referenced this pull request Mar 4, 2026
ghstack-source-id: fbb6fd0
Pull-Request: #176320
@malfet
Copy link
Contributor Author

malfet commented Mar 4, 2026

@pytorchbot merge -f "Lint is green, will monitor what will happen"

@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

@malfet
Copy link
Contributor Author

malfet commented Mar 4, 2026

@pytorchbot revert -m "It does not work, see https://github.com/pytorch/pytorch/actions/runs/22694439349/job/65797559931#step:10:26" -c weird

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

@pytorchmergebot
Copy link
Collaborator

@malfet your PR has been successfully reverted.

@pytorchmergebot pytorchmergebot added Reverted ci-no-td Do not run TD on this PR labels Mar 4, 2026
[ghstack-poisoned]
malfet added a commit that referenced this pull request Mar 4, 2026
ghstack-source-id: 1882617
Pull-Request: #176320
@malfet
Copy link
Contributor Author

malfet commented Mar 4, 2026

@pytorchbot merge -f "2nd time is the charm"

@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

@malfet
Copy link
Contributor Author

malfet commented Mar 5, 2026

@pytorchbot revert -m "It works now, but does not upload the log I need"

@pytorch-bot
Copy link

pytorch-bot bot commented Mar 5, 2026

❌ 🤖 pytorchbot command failed:

@pytorchbot revert: error: the following arguments are required: -c/--classification

usage: @pytorchbot revert -m MESSAGE -c
                          {nosignal,ignoredsignal,landrace,weird,ghfirst,autorevert}

Try @pytorchbot --help for more info.

@malfet
Copy link
Contributor Author

malfet commented Mar 5, 2026

@pytorchbot revert -m "It works now, but does not upload the log I need" -c weird

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

@pytorchmergebot
Copy link
Collaborator

@malfet your PR has been successfully reverted.

pytorchmergebot added a commit that referenced this pull request Mar 5, 2026
This reverts commit 41e5795.

Reverted #176320 on behalf of https://github.com/malfet due to It works now, but does not upload the log I need ([comment](#176320 (comment)))
[ghstack-poisoned]
malfet added a commit that referenced this pull request Mar 5, 2026
ghstack-source-id: b8b0a45
Pull-Request: #176320
@malfet
Copy link
Contributor Author

malfet commented Mar 5, 2026

@pytorchbot merge -f "Let's try it one more time"

@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

Vighaneshs pushed a commit to Vighaneshs/pytorch that referenced this pull request Mar 5, 2026
Vighaneshs pushed a commit to Vighaneshs/pytorch that referenced this pull request Mar 5, 2026
Vighaneshs pushed a commit to Vighaneshs/pytorch that referenced this pull request Mar 5, 2026
Vighaneshs pushed a commit to Vighaneshs/pytorch that referenced this pull request Mar 5, 2026
This reverts commit 41e5795.

Reverted pytorch#176320 on behalf of https://github.com/malfet due to It works now, but does not upload the log I need ([comment](pytorch#176320 (comment)))
Vighaneshs pushed a commit to Vighaneshs/pytorch that referenced this pull request Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-td Do not run TD on this PR Merged Reverted topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants