pytorch / executorch

On-device AI across mobile, embedded and edge for PyTorch
https://pytorch.org/executorch/
Other
1.31k stars 206 forks source link

Don't recurse submodules when building wheels #3469

Closed dbort closed 2 weeks ago

dbort commented 2 weeks ago

Stack from ghstack (oldest at bottom):

Change the build-wheels workflow to only fetch the first layer of submodules. ExecuTorch only needs the first layer of submodules to build its pip package, but the build_wheels_*.yaml workflows will recursively fetch all submodules by default.

Fetching all submodules can also cause buck2 to fail because it will try to watch too many files.

This change makes buck2 work on the CI runners, speeds up the jobs, and reduces disk/network usage.

Differential Revision: D56857487

pytorch-bot[bot] commented 2 weeks ago

:link: Helpful Links

:test_tube: See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/3469

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

:x: 2 New Failures

As of commit 914fb31f151c8f12a1991f39e1bb222a88307f07 with merge base 5d2a17b29f57ab77f0481ca1d96d827e610dc4bf (image):

NEW FAILURES - The following jobs have failed:

* [Build Linux Wheels / pytorch/executorch / upload / manywheel-py3_8-cpu](https://hud.pytorch.org/pr/pytorch/executorch/3469#24487083517) ([gh](https://github.com/pytorch/executorch/actions/runs/8916099106/job/24487083517)) `Unable to find any artifacts for the associated workflow` * [Build M1 Wheels / pytorch/executorch / upload / wheel-py3_8-cpu](https://hud.pytorch.org/pr/pytorch/executorch/3469#24487105691) ([gh](https://github.com/pytorch/executorch/actions/runs/8916099242/job/24487105691)) `Unable to find any artifacts for the associated workflow`

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

dbort commented 2 weeks ago

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

facebook-github-bot commented 2 weeks ago

@dbort merged this pull request in pytorch/executorch@f6b152785d08039c246a358cd0d0f92ba623d753.