Re-introduce some logging after backout #14526

metascroy · 2025-09-23T18:55:30Z

Summary:
Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps:

In diff 1/3, we backout D81256207, which added more logging
In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert
In diff 3/3, we introduce some of the logging we backed out in diff 1/3

Differential Revision: D82581441

pytorch-bot · 2025-09-23T18:55:34Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 4 New Failures, 1 Cancelled Job, 1 Pending, 1 Unrelated Failure

As of commit 16d27a4 with merge base 87c44c7 ():

NEW FAILURES - The following jobs have failed:

pull / test-qnn-wheel-packages-linux (3.10) / linux-job (gh)
RuntimeError: Command docker exec -t e3125a494809bacf1ac79b97b3cb231c4df13b36d75011a566e4a8190be2a15a /exec failed with exit code 1
pull / test-qnn-wheel-packages-linux (3.11) / linux-job (gh)
RuntimeError: Command docker exec -t 33d253217697e705fa8376d65a2a4c35668ee8335bc40bcc1cf4fd48636ab401 /exec failed with exit code 1
pull / test-qnn-wheel-packages-linux (3.12) / linux-job (gh)
RuntimeError: Command docker exec -t d223b05b7ba4c22c70c7d67e80c14969a7960a2d07f41444cad9d188a293d983 /exec failed with exit code 1
pull / test-samsung-models-linux / linux-job (gh)
RuntimeError: Command docker exec -t 19e5b8c324281de8b01cd313bfe811bf98a02d948c5e1d06216d61673889523b /exec failed with exit code 1

CANCELLED JOB - The following job was cancelled. Please retry:

pull / test-setup-linux-gcc / linux-job (gh)

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

pull / test-binary-size-linux-gcc / linux-job (gh) (trunk failure)
##[error]The operation was canceled.

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

facebook-github-bot · 2025-09-23T18:55:41Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

github-actions · 2025-09-23T18:56:19Z

This PR needs a `release notes:` label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Differential Revision: D82581441

facebook-github-bot · 2025-09-23T20:21:41Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

mergennachin · 2025-09-24T14:20:51Z

@metascroy did you figure out the root cause of the original revert?

Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Reviewed By: mergennachin Differential Revision: D82581441

facebook-github-bot · 2025-09-24T16:37:37Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Reviewed By: mergennachin Differential Revision: D82581441

facebook-github-bot · 2025-09-24T17:03:18Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

facebook-github-bot · 2025-09-24T17:03:26Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Reviewed By: mergennachin Differential Revision: D82581441

facebook-github-bot · 2025-09-24T20:25:04Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Reviewed By: mergennachin Differential Revision: D82581441

facebook-github-bot · 2025-09-24T20:26:39Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Reviewed By: mergennachin Differential Revision: D82581441

facebook-github-bot · 2025-09-25T16:24:00Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Reviewed By: mergennachin Differential Revision: D82581441

facebook-github-bot · 2025-09-25T16:24:33Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

Summary: Since D80715432 landed, we also landed D81256207 to add more logging. To prevent conflicts, we backout in 3 steps: * In diff 1/3, we backout D81256207, which added more logging * In diff 2/3, we backout D80715432, which is the problematic diff we're trying to revert * In diff 3/3, we introduce some of the logging we backed out in diff 1/3 Reviewed By: mergennachin Differential Revision: D82581441

facebook-github-bot · 2025-09-25T16:55:32Z

@metascroy has exported this pull request. If you are a Meta employee, you can view the originating diff in D82581441.

metascroy · 2025-09-25T20:22:04Z

@pytorchbot cherry-pick --onto release/1.0 -c critical

metascroy · 2025-09-25T20:23:33Z

@metascroy did you figure out the root cause of the original revert?

Still investigating with internal teams. I do not think there was an issue with the OSS builds, though.

Differential Revision: D82581441 Pull Request resolved: #14526 (cherry picked from commit 32b87e0)

pytorchbot · 2025-09-25T20:24:24Z

Cherry picking #14526

The cherry pick PR is at #14612 and it is recommended to link a critical cherry pick PR with an issue. The following tracker issues are updated:

[v1.0.0] Release Tracker #14288 (comment)

Details for Dev Infra team

Raised by workflow job

Differential Revision: D82581441 Pull Request resolved: #14526 (cherry picked from commit 32b87e0)

metascroy requested review from cccclai and shoumikhin as code owners September 23, 2025 18:55

meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 23, 2025

facebook-github-bot added fb-exported meta-exported labels Sep 23, 2025

metascroy added the ciflow/trunk label Sep 23, 2025

metascroy force-pushed the export-D82581441 branch from ca8a22e to 944df24 Compare September 23, 2025 20:21

mergennachin approved these changes Sep 24, 2025

View reviewed changes

metascroy force-pushed the export-D82581441 branch from 944df24 to c278f0c Compare September 24, 2025 16:37

metascroy force-pushed the export-D82581441 branch from c278f0c to 95dc2f1 Compare September 24, 2025 17:03

metascroy force-pushed the export-D82581441 branch from 95dc2f1 to 48dd5a5 Compare September 24, 2025 17:03

metascroy force-pushed the export-D82581441 branch from 48dd5a5 to 0975ebd Compare September 24, 2025 20:24

metascroy force-pushed the export-D82581441 branch from 0975ebd to 63537c6 Compare September 24, 2025 20:26

metascroy force-pushed the export-D82581441 branch from 63537c6 to 9971d35 Compare September 25, 2025 16:23

metascroy force-pushed the export-D82581441 branch from 9971d35 to 1337887 Compare September 25, 2025 16:24

metascroy force-pushed the export-D82581441 branch from 1337887 to 16d27a4 Compare September 25, 2025 16:55

facebook-github-bot merged commit 32b87e0 into pytorch:main Sep 25, 2025
279 of 286 checks passed

pytorchbot pushed a commit that referenced this pull request Sep 25, 2025

Re-introduce some logging after backout

639a834

Differential Revision: D82581441 Pull Request resolved: #14526 (cherry picked from commit 32b87e0)

pytorchbot mentioned this pull request Sep 25, 2025

[v1.0.0] Release Tracker #14288

Closed

GregoryComer pushed a commit that referenced this pull request Sep 29, 2025

Re-introduce some logging after backout

ff436e9

Differential Revision: D82581441 Pull Request resolved: #14526 (cherry picked from commit 32b87e0)

Re-introduce some logging after backout #14526

Re-introduce some logging after backout #14526

Uh oh!

Conversation

metascroy commented Sep 23, 2025

Uh oh!

pytorch-bot bot commented Sep 23, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 Helpful Links

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

❌ 4 New Failures, 1 Cancelled Job, 1 Pending, 1 Unrelated Failure

Uh oh!

facebook-github-bot commented Sep 23, 2025

Uh oh!

github-actions bot commented Sep 23, 2025

This PR needs a release notes: label

Uh oh!

facebook-github-bot commented Sep 23, 2025

Uh oh!

mergennachin commented Sep 24, 2025

Uh oh!

facebook-github-bot commented Sep 24, 2025

Uh oh!

facebook-github-bot commented Sep 24, 2025

Uh oh!

facebook-github-bot commented Sep 24, 2025

Uh oh!

facebook-github-bot commented Sep 24, 2025

Uh oh!

facebook-github-bot commented Sep 24, 2025

Uh oh!

facebook-github-bot commented Sep 25, 2025

Uh oh!

facebook-github-bot commented Sep 25, 2025

Uh oh!

facebook-github-bot commented Sep 25, 2025

Uh oh!

Uh oh!

metascroy commented Sep 25, 2025

Uh oh!

metascroy commented Sep 25, 2025

Uh oh!

pytorchbot commented Sep 25, 2025

Cherry picking #14526

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

pytorch-bot bot commented Sep 23, 2025 •

edited

Loading

This PR needs a `release notes:` label