Skip to content

fix(container-stack): repair clean-build breakages + add podman smoke CI#37

Merged
hyperpolymath merged 2 commits into
mainfrom
claude/resolve-open-issues-29d7x
May 15, 2026
Merged

fix(container-stack): repair clean-build breakages + add podman smoke CI#37
hyperpolymath merged 2 commits into
mainfrom
claude/resolve-open-issues-29d7x

Conversation

@hyperpolymath
Copy link
Copy Markdown
Owner

Closes #17.

Context

#17 reported that four of the five container-stack/ Containerfiles failed on a clean build. Auditing the current tree:

Component #17 issue State on this branch
cerro-torre Alire URL 404 (#13) Already fixed (explicit-version URL); plus a remaining COPY config/ config/ clean-build break, fixed here
vordr Cargo.lock not committed Lockfile now tracked, but .gitignore still ignored it (latent regression) — fixed here
svalinn npm/rescript-vs-Deno layout Already rewritten for Deno + rescript
rokur un-parseable Deno install URL Already rewritten to direct binary download
selur Containerfile missing (#12) Already added

Changes

  • vordr/.gitignore — stop ignoring Cargo.lock. vordr ships a binary crate and its Containerfile builds with cargo build --locked; the ignore rule was the root cause of the "Cargo.lock not committed" failure and would silently re-drop the lockfile on any regeneration.
  • cerro-torre/Containerfile — drop COPY config/ config/. config/ is Alire-generated and gitignored, so it is absent on a fresh clone and the COPY broke the build before alr build (which regenerates it) ever ran.
  • .github/workflows/container-stack-smoke.yml — matrix podman build of every container-stack/ Containerfile on changes to that tree. This is the durable guard [meta] Wave 1 container-stack: Containerfiles need maintenance pass #17 asks for: every one of the four bug classes lives in an early build step, so a real podman build catches the whole class on every push. cerro-torre's full alr build depends on the un-vendored upstream proven Ada library (a concern separate from [meta] Wave 1 container-stack: Containerfiles need maintenance pass #17), so it is built non-blocking until proven is available while still exercising and logging the Alire-download step.

Test plan

  • container-stack smoke build workflow runs and the vordr, svalinn, rokur, selur jobs go green
  • cerro-torre job runs, exercises the Alire download step, and is reported non-blocking
  • Workflow Security Linter passes (SPDX header, top-level permissions:, SHA-pinned action all satisfied)

https://claude.ai/code/session_014cznZXkqptPSoZDFhp7bhc


Generated by Claude Code

claude added 2 commits May 15, 2026 11:29
Resolves the maintenance pass tracked in stapeln#17.

- vordr/.gitignore: stop ignoring Cargo.lock. The binary crate's lockfile
  must stay tracked (Containerfile builds with `cargo build --locked`);
  the ignore rule was the root cause of the "Cargo.lock not committed"
  clean-build failure.
- cerro-torre/Containerfile: drop `COPY config/ config/`. config/ is
  Alire-generated and gitignored, so it is absent on a fresh clone and
  the COPY broke the build; `alr build` regenerates it.
- Add .github/workflows/container-stack-smoke.yml: matrix `podman build`
  of every container-stack Containerfile so the whole class of early
  build-step regressions is gated on every push. cerro-torre's full Ada
  build depends on the un-vendored upstream `proven` library and is
  built non-blocking until that is available.

https://claude.ai/code/session_014cznZXkqptPSoZDFhp7bhc
GitHub's ubuntu-latest image no longer ships podman preinstalled, so the
smoke jobs failed in ~14s at the `podman version` step. Install podman
via apt before building.

https://claude.ai/code/session_014cznZXkqptPSoZDFhp7bhc
@sonarqubecloud
Copy link
Copy Markdown

@hyperpolymath hyperpolymath marked this pull request as ready for review May 15, 2026 11:31
@hyperpolymath hyperpolymath merged commit c86ee1f into main May 15, 2026
29 of 40 checks passed
@hyperpolymath hyperpolymath deleted the claude/resolve-open-issues-29d7x branch May 15, 2026 11:32
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.

[meta] Wave 1 container-stack: Containerfiles need maintenance pass

2 participants