Repository navigation
fix(cuda): advance the layer-nvidia pin past the Arch gpgcheck fix (#12) - #16
Merged
Merged
Conversation
The transitive pin won the closure resolution over a consumer's direct pin, so every consumer composing cuda inherited the pre-fix nvidia check and its check-image bed failed on Arch with the removed stdout matcher: 'stdout: expected to contain gpgcheck=1 (got: )'. Measured on an Arch host in opencharly/layer-strata, whose own candy pins layer-nvidia at the merged fix and still ran the pre-fix check. layer-nvidia#11/#12 removed that matcher, moving the assertion into the command where the distro decision is made; v2026.271.1817 predates it. Assisted-by: DeepSeek Harness deepseek-v4.1-flash (syntax check only)
Review — PASSHead SHA: Security screen: PASS — one pin line to the org's own Verdict: PASS ollama-cloud/deepseek-v4.1-flash — action-review. |
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.
Summary
Advances this repo's
layer-nvidiapin offv2026.271.1817and onto the merge commit ofopencharly/layer-nvidia#12(054aa7702ebe79e8be444764f91a03933ba45eec) — the change that moved thenvidia-container-toolkitgpgcheck assertion into the command so the Arch arm can skip honestly instead offailing. One line in the root
charly.yml; no other pin moves.The stale pin is real and was re-measured against the API rather than recalled:
mainpinnedv2026.271.1817while the fix merged around CalVer day 289. It is visible from the consumer side —
opencharly/layer-strata'scheck-strata-gpu-podbuilds a tree whose transitivelayer-nvidiais that old pin, so the bed runs thepre-#12 probe and fails on
nvidia-container-toolkit-gpgcheck-enforced.A merged SHA is used rather than the tag, deliberately: it is an immutable ref, and
charly box reconcilealready reports this tree as aligned ("every repo's pins are at one version").
How tested
charly box validate— the class pre-flight for a manifest change: OK, 0 warnings, 0 errors.charly box reconcile --dry-runin this tree: "already reconciled — every repo's pins are at one version."check-strata-gpu-pod(the consumer's real-engine GPU bed) — named, with its measured result: a full runtoday built the cachyos+CUDA+engine image (PASS after 17m56s,
strata-engine-binary,strata-engine-stamp,strata-venvand the whole CUDA chain all PASS) and then failed the container phase on exactly one step:nvidia-container-toolkit-gpgcheck-enforced— the pre-The Arch stitch never links the STATIC CUDA archives, so every CUDA link fails (cannot find -lcudart_static) #12 probe. Its 12 runtime steps (including the completionround-trip) were skipped behind that failure. This pin advance is the fix for that step; the bed re-run
follows this merge and will be reported here.
So: the gate that this repo's change class requires has run and passed; the behavioural proof that depends on the
advance cannot run until it is merged, and that is stated rather than implied.
Rulebook compliance
/charly-build:reconcile,/charly-internals:layer,/charly-core:deployand/charly-internals:git-workflowloaded before any edit or git action.#14/#15(an invalid control: 32 stale
layer-nvidiaref views in~/.cache/charly/repos/made a "still fails" readinglook like disproof). That cache has since been cleared on the bed host, so the next resolution is fresh.
git statusclean before and after; PR-only; no force-push; the branch is this session's own slug's.Change classification
Pinning / Policy B (Part II #3): a pin advance to a merged ref, landed PR-only. Not a behaviour change in this
repo's own contents — the behaviour change is the producer's, already merged and tagged in
layer-nvidia.Filed issues
opencharly/layer-cuda#14— closes: this is the fix that issue asks for ("stale pin, ~18 days behind").opencharly/layer-nvidia#11— the producer-side defect this pin carries the fix for; its probe now passes onmainand the issue can be closed once this consumer resolves the new ref.opencharly/layer-cuda#15was closed on an invalid control (see R1 above). This PR is itssuccessor per Part II The cuda candy pins cuda-cccl-devel, which the NVIDIA Fedora 43 repo does not provide — every GPU image fails to build #7;
#15is superseded, not silently replaced.Assisted-by: DeepSeek Harness deepseek-v4.1-flash (analysed on a live system)