Skip to content

feat: add extra-no-build-isolation-packages input to wheel build workflow#440

Merged
ko3n1g merged 1 commit intomainfrom
ko3n1g/fix/extra-pre-install-packages
Apr 1, 2026
Merged

feat: add extra-no-build-isolation-packages input to wheel build workflow#440
ko3n1g merged 1 commit intomainfrom
ko3n1g/fix/extra-pre-install-packages

Conversation

@ko3n1g
Copy link
Copy Markdown
Contributor

@ko3n1g ko3n1g commented Apr 1, 2026

Summary

  • Some packages (e.g. nvidia-resiliency-ext 0.6.0) need build-time access to packages that are only declared as runtime deps (not in build-system.requires), so uv doesn't know to pre-install them before the build
  • The no-build-isolation pre-install step is hardcoded to torch setuptools pybind11 wheel_stub packaging poetry poetry-dynamic-versioning
  • This PR adds an extra-no-build-isolation-packages input that lets callers inject additional packages into that pre-install step

Example

uses: NVIDIA-NeMo/FW-CI-templates/.github/workflows/_build_test_publish_wheel.yml@v0.87.0
with:
  no-build-isolation: true
  extra-no-build-isolation-packages: "grpcio-tools"

Test plan

🤖 Generated with Claude Code

…flow

Some packages (e.g. nvidia-resiliency-ext 0.6.0) require additional
packages (e.g. grpcio-tools) to be available at build time, but do not
declare them in build-system.requires. The new input lets callers
pre-install these packages before the --no-build-isolation install runs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: oliver könig <okoenig@nvidia.com>
ko3n1g added a commit to NVIDIA-NeMo/Megatron-Bridge that referenced this pull request Apr 1, 2026
nvidia-resiliency-ext 0.6.0 compiles proto files at build time via
grpc_tools.protoc, but grpcio-tools is only a runtime dep and is not
pre-installed by the --no-build-isolation setup. Adding it via the new
extra-no-build-isolation-packages template input ensures it is available
before the build runs.

Bumps FW-CI-templates to 5e47529 (NVIDIA-NeMo/FW-CI-templates#440).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: oliver könig <okoenig@nvidia.com>
ko3n1g added a commit to NVIDIA-NeMo/Megatron-Bridge that referenced this pull request Apr 1, 2026
nvidia-resiliency-ext 0.6.0 compiles proto files at build time via
grpc_tools.protoc, but grpcio-tools is only a runtime dep and is not
pre-installed by the --no-build-isolation setup. Adding it via the new
extra-no-build-isolation-packages template input ensures it is available
before the build runs.

Bumps FW-CI-templates to 5e47529 (NVIDIA-NeMo/FW-CI-templates#440).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: oliver könig <okoenig@nvidia.com>
ko3n1g added a commit to NVIDIA-NeMo/Megatron-Bridge that referenced this pull request Apr 1, 2026
nvidia-resiliency-ext 0.6.0 compiles proto files at build time via
grpc_tools.protoc, but grpcio-tools is only a runtime dep and is not
pre-installed by the --no-build-isolation setup. Adding it via the new
extra-no-build-isolation-packages template input ensures it is available
before the build runs.

Bumps FW-CI-templates to 5e47529 (NVIDIA-NeMo/FW-CI-templates#440).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: oliver könig <okoenig@nvidia.com>
ko3n1g added a commit to NVIDIA-NeMo/Megatron-Bridge that referenced this pull request Apr 1, 2026
nvidia-resiliency-ext 0.6.0 compiles proto files at build time via
grpc_tools.protoc, but grpcio-tools is only a runtime dep and is not
pre-installed by the --no-build-isolation setup. Adding it via the new
extra-no-build-isolation-packages template input ensures it is available
before the build runs.

Bumps FW-CI-templates to 5e47529 (NVIDIA-NeMo/FW-CI-templates#440).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: oliver könig <okoenig@nvidia.com>
@ko3n1g ko3n1g merged commit 833caa8 into main Apr 1, 2026
5 checks passed
@ko3n1g ko3n1g deleted the ko3n1g/fix/extra-pre-install-packages branch April 1, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants