migrate examples/models/llama/config (fbcode-only python_unittest) by bigfootjon · Pull Request #21428 · pytorch/executorch

bigfootjon · 2026-07-28T16:15:47Z

Summary: Add is_fbcode early-return; replace TARGETS with thin BUCK.

Differential Revision: D109082042

Summary: Chunk 11 of fbcode/executorch TARGETS->BUCK migration. 5 directories that had only a TARGETS file with substantive fbcode-only content but were skipped from earlier chunks for various reasons (`fbcode_target(`, `fbcode_macros`, fbcode-only kwargs, etc.). Same chunk-6 pattern: created `targets.bzl` with `define_common_targets(is_fbcode = False)` whose body opens with `if not is_fbcode: return`, replaced TARGETS with a thin BUCK calling `define_common_targets(is_fbcode = is_fbcode())`. Directories migrated: - backends/aoti/tests - backends/cuda/runtime - backends/cuda/tests - extension/pybindings/fb - extension/pytree/test Differential Revision: D109082053

Summary: Add is_fbcode param to define_common_targets and early-return outside fbcode. The cxx_test rules use fbcode-only kwargs (keep_gpu_sections, re_test_utils.remote_execution) and were never evaluated in xplat (TARGETS-only pre-migration). Replaced TARGETS with thin BUCK calling define_common_targets(is_fbcode = is_fbcode()). Differential Revision: D109082041

Summary: Add is_fbcode early-return to existing define_common_targets. The cpp_unittest rules use fbcode-only kwargs (keep_gpu_sections, re_test_utils.remote_execution) and can't run in xplat. Replaced TARGETS with thin BUCK. Differential Revision: D109082049

Summary: Add is_fbcode early-return; replace TARGETS with thin BUCK. Differential Revision: D109082042

pytorch-bot · 2026-07-28T16:15:51Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 2 New Failures

As of commit ee91e7d with merge base c7b6f52 ():

NEW FAILURES - The following jobs have failed:

Cadence Build & Test / hifi-build / hifi4 (gh)
Refusing to check out fork pull request code from a 'pull_request_target' workflow. This workflow runs with the base repository's GITHUB_TOKEN, secrets, default-branch cache scope, and runner access. Fetching and executing a fork's code in that trusted context commonly leads to "pwn request" vulnerabilities. To opt in, review the risks at https://gh.io/securely-using-pull_request_target and set 'allow-unsafe-pr-checkout: true' on the actions/checkout step.
Cadence Build & Test / vision-build / vision (gh)
Refusing to check out fork pull request code from a 'pull_request_target' workflow. This workflow runs with the base repository's GITHUB_TOKEN, secrets, default-branch cache scope, and runner access. Fetching and executing a fork's code in that trusted context commonly leads to "pwn request" vulnerabilities. To opt in, review the risks at https://gh.io/securely-using-pull_request_target and set 'allow-unsafe-pr-checkout: true' on the actions/checkout step.

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

meta-codesync · 2026-07-28T16:16:47Z

@bigfootjon has exported this pull request. If you are a Meta employee, you can view the originating Diff in D109082042.

github-actions · 2026-07-28T16:17:59Z

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.

Copilot

Pull request overview

This PR migrates several fbcode Buck target definitions from legacy TARGETS files into thin BUCK entrypoints that delegate to targets.bzl, adding an is_fbcode gate to preserve prior behavior when these definitions are evaluated outside fbcode.

Changes:

Replace multiple TARGETS files with BUCK files that call define_common_targets(is_fbcode = is_fbcode()).
Introduce new targets.bzl files (or update existing ones) to host the shared target definitions.
Add early-return gating in define_common_targets to avoid defining fbcode-only targets when not in fbcode.

Reviewed changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
extension/pytree/test/targets.bzl	New `define_common_targets(is_fbcode=...)` with fbcode-gated C++/Python unittests.
extension/pytree/test/TARGETS	Removed legacy TARGETS definition in favor of BUCK indirection.
extension/pytree/test/BUCK	New thin BUCK that calls into `targets.bzl` with `is_fbcode()`.
examples/models/llama/config/targets.bzl	Gate `python_unittest` target definition behind `is_fbcode`.
examples/models/llama/config/TARGETS	Removed legacy TARGETS entrypoint.
examples/models/llama/config/BUCK	New BUCK entrypoint calling `define_common_targets(is_fbcode = is_fbcode())`.
backends/cuda/tests/targets.bzl	New fbcode-gated Python unittest targets for CUDA backend tests.
backends/cuda/tests/TARGETS	Removed legacy TARGETS definitions.
backends/cuda/tests/BUCK	New BUCK entrypoint calling into `targets.bzl`.
backends/cuda/runtime/targets.bzl	New fbcode-gated runtime libraries and CUDA C++ unittests migrated from TARGETS.
backends/cuda/runtime/TARGETS	Removed legacy TARGETS definitions.
backends/cuda/runtime/shims/tests/targets.bzl	Add `is_fbcode` gating to preserve pre-migration behavior for shim tests.
backends/cuda/runtime/shims/tests/TARGETS	Removed legacy TARGETS entrypoint.
backends/cuda/runtime/shims/tests/BUCK	New BUCK entrypoint calling into `targets.bzl`.
backends/cuda/runtime/BUCK	New BUCK entrypoint calling into `targets.bzl`.
backends/aoti/tests/targets.bzl	New fbcode-gated AOTI C++ unittest targets migrated from TARGETS.
backends/aoti/tests/TARGETS	Removed legacy TARGETS definitions.
backends/aoti/tests/BUCK	New BUCK entrypoint calling into `targets.bzl`.
backends/aoti/slim/factory/test/targets.bzl	Add `is_fbcode` gating around existing SlimTensor factory tests.
backends/aoti/slim/factory/test/TARGETS	Removed legacy TARGETS entrypoint.
backends/aoti/slim/factory/test/BUCK	New BUCK entrypoint calling into `targets.bzl`.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+load("@fbsource//xplat/executorch/build:runtime_wrapper.bzl", "runtime")
+load("@fbcode_macros//build_defs:python_unittest.bzl", "python_unittest")
+load("@fbcode_macros//build_defs:python_unittest_remote_gpu.bzl", "python_unittest_remote_gpu")
+load("@fbcode_macros//build_defs/lib:re_test_utils.bzl", "re_test_utils")


+        deps = [
+            ":cuda_platform",
+            ":runtime_shims",
+            ":cuda_allocator",
+            ":cuda_platform",


+            "//executorch/runtime/core/exec_aten:lib",
+            "//executorch/extension/tensor:tensor",
+        ],


ndmitchell

Review automatically exported from Phabricator review in Meta.

bigfootjon added 4 commits July 28, 2026 08:55

migrate examples/models/llama/config (fbcode-only python_unittest)

ee91e7d

Summary: Add is_fbcode early-return; replace TARGETS with thin BUCK. Differential Revision: D109082042

Copilot AI review requested due to automatic review settings July 28, 2026 16:15

bigfootjon requested a review from JacobSzwejbka as a code owner July 28, 2026 16:15

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 Jul 28, 2026

Copilot started reviewing on behalf of bigfootjon July 28, 2026 16:16 View session

meta-codesync Bot added the meta-exported label Jul 28, 2026

meta-codesync Bot had a problem deploying to cadence July 28, 2026 16:16 Failure

Copilot AI reviewed Jul 28, 2026

View reviewed changes

ndmitchell approved these changes Jul 28, 2026

View reviewed changes

SS-JIA approved these changes Jul 29, 2026

View reviewed changes

meta-codesync Bot merged commit f4a21b8 into pytorch:main Jul 29, 2026
231 of 242 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

migrate examples/models/llama/config (fbcode-only python_unittest) - #21428

migrate examples/models/llama/config (fbcode-only python_unittest)#21428
meta-codesync[bot] merged 4 commits into
pytorch:mainfrom
bigfootjon:export-D109082042

bigfootjon commented Jul 28, 2026

Uh oh!

pytorch-bot Bot commented Jul 28, 2026 •

edited

Loading

Uh oh!

meta-codesync Bot commented Jul 28, 2026

Uh oh!

github-actions Bot commented Jul 28, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

ndmitchell left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

bigfootjon commented Jul 28, 2026

Uh oh!

pytorch-bot Bot commented Jul 28, 2026 • 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/21428

❌ 2 New Failures

Uh oh!

meta-codesync Bot commented Jul 28, 2026

Uh oh!

github-actions Bot commented Jul 28, 2026

This PR needs a release notes: label

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

ndmitchell left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

pytorch-bot Bot commented Jul 28, 2026 •

edited

Loading

This PR needs a `release notes:` label