🔗 Helpful Links

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

• 📄 Preview Python docs built from this PR
• 📄 Preview C++ docs built from this PR
• ❓ Need help or want to give feedback on the CI? Visit the bot commands wiki

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

❌ 2 New Failures, 6 Unrelated Failures

As of commit d340655 with merge base 8bbabb0 ():

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

This PR needs a release notes: label

If your changes are user facing and intended to be a part of release notes, please use a label starting with release notes:.

If not, please add the topic: not user facing label.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "topic: not user facing"

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

@pytorchmergebot merge -i

Merge started

Your change will be merged while ignoring the following 3 checks: operator_benchmark / aarch64-opbenchmark-test / test (cpu_operator_benchmark_short, 1, 1, linux.arm64.m8g.4xlarge), operator_benchmark / x86-opbenchmark-test / test (cpu_operator_benchmark_short, 1, 1, linux.12xlarge), inductor / inductor-cpu-test / test (cpu_inductor_torchbench, 1, 2, linux.2xlarge.amx, unstable)

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

@pytorchbot revert -m "Reverted automatically by pytorch's autorevert, to avoid this behaviour add the tag autorevert: disable" -c autorevert

This PR is attributed to have caused regression in:

• vllm-test: vllm-x-pytorch-build / build (hud)

Please investigate and fix the issues.
@claude Can you please read this revert comment, follow the links and read the errors, to then give a brief diagnostics on the cause of the error? If you judge the error to be legitimate reason for a revert, please provide brief guidance on how the author could fix it.

Claude finished @pytorch-auto-revert[bot]'s task in 1m 50s —— View job

Investigating Revert of PR #176458

• Read the CI failure logs from the vllm-test build
• Identify the root cause of the failure
• Assess if the revert is legitimate
• Provide guidance on fixing the issue

Diagnosis

The revert is legitimate. The root cause is a Python version incompatibility.

The PR converted Optional[X] to X | None (PEP 604 union syntax) across many files, including the .ci/lumen_cli/ directory. The problem is that the vllm CI build environment runs the lumen_cli tool using Python 3.9 (the system Python on the CI runner), not Python 3.12 (which is used for the actual PyTorch build).

The X | None union syntax in type annotations at module scope (not inside from __future__ import annotations or string annotations) requires Python 3.10+ at runtime. When Python 3.9 tries to evaluate docker.DockerClient | None at line 14 of docker_helper.py, it fails with:

TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

The full traceback from the CI log:

File ".ci/lumen_cli/cli/lib/common/docker_helper.py", line 14, in <module>
    _docker_client: docker.DockerClient | None = None
TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

How to Fix

The .ci/lumen_cli/ files must retain the Optional[X] / Union[X, Y] syntax (or use from __future__ import annotations at the top of each file) because they are executed with Python 3.9 in the vllm CI environment. Specifically, the following files from this PR need to be reverted or have from __future__ import annotations added:

• .ci/lumen_cli/cli/lib/common/docker_helper.py
• .ci/lumen_cli/cli/lib/common/envs_helper.py
• .ci/lumen_cli/cli/lib/common/path_helper.py
• .ci/lumen_cli/cli/lib/common/pip_helper.py
• .ci/lumen_cli/cli/lib/common/utils.py
• .ci/lumen_cli/cli/lib/core/vllm/vllm_build.py

The simplest fix is to add from __future__ import annotations to each of these files, which makes all annotations strings (lazy evaluation) and avoids the runtime TypeError. Alternatively, exclude the .ci/lumen_cli/ directory from the ruff UP007/UP045 rules, or revert just those files back to Optional/Union syntax.

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

@Lucaskabela your PR has been successfully reverted.

@pytorchmergebot merge

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

The merge job was canceled or timed out. This most often happen if two merge requests were issued for the same PR, or if merge job was waiting for more than 6 hours for tests to finish. In later case, please do not hesitate to reissue the merge command
For more information see pytorch-bot wiki.

@pytorchmergebot rebase

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

Successfully rebased lucaskabela/up007_up045_ci_tools onto refs/remotes/origin/viable/strict, please pull locally before adding more changes (for example, via git checkout lucaskabela/up007_up045_ci_tools && git pull --rebase)

@pytorchmergebot merge

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

Merge failed

Reason: 1 jobs have failed, first few of them are: operator_benchmark / aarch64-opbenchmark-test / test (cpu_operator_benchmark_short, 1, 1, linux.arm64.m8g.4xlarge)

@pytorchmergebot merge -i

Merge started

Your change will be merged while ignoring the following 8 checks: operator_benchmark / aarch64-opbenchmark-test / test (cpu_operator_benchmark_short, 1, 1, linux.arm64.m8g.4xlarge), operator_benchmark / x86-opbenchmark-test / test (cpu_operator_benchmark_short, 1, 1, linux.12xlarge), linux-aarch64 / linux-jammy-aarch64-py3.10 / test (openreg, 1, 1, lf.linux.arm64.m7g.4xlarge), inductor / inductor-cpu-test / test (cpu_inductor_torchbench, 1, 2, linux.2xlarge.amx, unstable), inductor / unit-test / inductor-test / test (inductor_cpp_wrapper, 1, 2, linux.g5.4xlarge.nvidia.gpu), inductor / unit-test / inductor-test / test (inductor_cpp_wrapper, 2, 2, linux.g5.4xlarge.nvidia.gpu), inductor / unit-test / inductor-test / test (inductor, 2, 2, linux.g5.4xlarge.nvidia.gpu), inductor / inductor-test / test (inductor_torchbench, 2, 2, linux.g5.4xlarge.nvidia.gpu)

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team