Skip to content

CI: skip default-packages on every macos_job.yml callsite#19297

Merged
JacobSzwejbka merged 1 commit intopytorch:mainfrom
rascani:skip-default-packages-on-macos-jobs
May 5, 2026
Merged

CI: skip default-packages on every macos_job.yml callsite#19297
JacobSzwejbka merged 1 commit intopytorch:mainfrom
rascani:skip-default-packages-on-macos-jobs

Conversation

@rascani
Copy link
Copy Markdown
Contributor

@rascani rascani commented May 5, 2026

Summary

pytorch/test-infra's setup-miniconda action pre-installs cmake=3.22 ninja=1.10 pkg-config=0.29 wheel=0.37 from the anaconda defaults channel into the conda env it sets up for macOS jobs. Our own setup-conda.sh then installs cmake=3.31.2 and friends from conda-forge into the same env, and reconciling the two channels' transitive deps (e.g. zlib=1.2.13 vs libzlib>=1.3.1, rhash=1.4.3 vs rhash>=1.4.5) has been intermittently failing the libmamba solver.

The companion test-infra PR exposes a default-packages input on macos_job.yml. Pass an empty string from every macos_job.yml callsite in this repo so the conda env created by setup-miniconda no longer pre-pollutes the env with defaults-channel packages we don't use, and our subsequent conda-forge install resolves cleanly.

This change has no effect until the test-infra PR lands. Once it's merged on test-infra@main, the workflows here pick it up automatically because executorch tracks @main for all test-infra references.

Authored with Claude Code.

Test plan

CI

pytorch/test-infra's setup-miniconda action pre-installs
cmake=3.22 ninja=1.10 pkg-config=0.29 wheel=0.37 from the anaconda
defaults channel into the conda env it sets up for macOS jobs. Our
own setup-conda.sh then installs cmake=3.31.2 and friends from
conda-forge into the same env, and reconciling the two channels'
transitive deps (e.g. zlib=1.2.13 vs libzlib>=1.3.1, rhash=1.4.3 vs
rhash>=1.4.5) has been intermittently failing the libmamba solver.

The companion test-infra PR exposes a default-packages input on
macos_job.yml. Pass an empty string from every macos_job.yml callsite
in this repo so the conda env created by setup-miniconda no longer
pre-pollutes the env with defaults-channel packages we don't use, and
our subsequent conda-forge install resolves cleanly.

This change has no effect until the test-infra PR lands. Once it's
merged on test-infra@main, the workflows here pick it up automatically
because executorch tracks @main for all test-infra references.

Authored with Claude Code.
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented May 5, 2026

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

⏳ No Failures, 17 Pending

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

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

@meta-cla 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 May 5, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

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.

@JacobSzwejbka JacobSzwejbka merged commit acffcb0 into pytorch:main May 5, 2026
28 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants