Skip to content

fix: accept bundled native runtime licenses - #28

Merged
i386 merged 1 commit into
mainfrom
fix/cuda-runtime-license-entry
Sep 12, 2026
Merged

i386 merged 1 commit into
mainfrom
fix/cuda-runtime-license-entry

Conversation

@i386

@i386 i386 commented Sep 12, 2026

Copy link
Copy Markdown

The v0.76.1 producer now bundles NVIDIA redistribution terms under each Linux CUDA runtime. The packaging verifier rejected those otherwise valid archives because its strict path allowlist did not include licenses/, blocking native packages and images after the public release.

Accept regular files below the runtime licenses/ directory while retaining the existing checksum, safe-entry-type, product-manifest, runtime-tree-digest, and single-runtime validation. Add a focused fixture for the exact NVIDIA-CUDA-LICENSE.txt path.

Validation:

  • node --experimental-strip-types scripts/image-matrix.ts validate
  • representative v0.76.1 CUDA and upstream matrix expansion
  • full TypeScript suite: 161 passed, 1 Docker-only skipped
  • image-matrix coverage: 100% lines, branches, and functions
  • actionlint -color
  • git diff --check

@coderabbitai

coderabbitai Bot commented Sep 12, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: a50ce247-91f6-4da7-b5af-eff883549ffb


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@i386
i386 merged commit fb4e29b into main Sep 12, 2026
2 checks passed
@i386
i386 deleted the fix/cuda-runtime-license-entry branch September 12, 2026 10:50
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.

1 participant