feat(uv): sample wheel build memory usage when resource_set is set#1140
Open
xangcastle wants to merge 7 commits into
Open
feat(uv): sample wheel build memory usage when resource_set is set#1140xangcastle wants to merge 7 commits into
xangcastle wants to merge 7 commits into
Conversation
✨ Aspect Workflows Tasks📅 Tue Jun 23 05:31:20 UTC 2026 ✅ 7 successful tasks
⏱ Last updated Tue Jun 23 05:34:09 UTC 2026 · 📊 GitHub API quota 105/15,000 (1% used, resets in 30m) |
cbbab78 to
f1048c6
Compare
…ry' into xangcastle/uv-1-wheel-build-memory # Conflicts: # docs/uv-patching.md # e2e/BUILD.bazel # e2e/cases/uv-sdist-jdk-build/setup.MODULE.bazel # uv/private/extension/defs.bzl # uv/private/pep517_whl/BUILD.bazel # uv/private/pep517_whl/rule.bzl # uv/private/sdist_build/repository.bzl
tamird
referenced
this pull request
Jun 23, 2026
Let each sdist override declare a local execution memory estimate. Bazel can then limit concurrent wheel builds without reserving the same amount for every package. When an estimate is present, sample aggregate RSS for the build process tree and print the peak on exit. The action log supplies evidence for adjusting that package's estimate after a successful build or OOM.
Contributor
|
The
The output should remain explicitly approximate Linux procfs RSS, not a hard |
Contributor
|
Superseded by #1174. The replacement is rebuilt on current |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a memory-monitoring layer to sdist wheel builds. When a
resource_setother than"default"is declared, the wheel build action now runs under a sampler that tracks aggregate resident-set size (RSS) of the build process and all its descendants, and reports the peak to stderr.This gives users concrete data to calibrate their
resource_setestimates. Rather than guessing whether"mem_2g"is enough, they can observe what the build actually consumes.Changes are visible to end-users: yes
Test plan