Skip to content

ci: build once, gate the digest, promote what was tested (RT#1550) - #31

Merged
fatherlinux merged 3 commits into
mainfrom
feat/container-release
Nov 11, 2026
Merged

fatherlinux merged 3 commits into
mainfrom
feat/container-release

Conversation

@fatherlinux

Copy link
Copy Markdown
Member

Summary

Part of RT#1550, the ubi10 chain, following the ubi10-core pilot (ubi10-core#37).

  • build.yml becomes container.yml, which calls constitution's [email protected].
  • The gates run against the pushed digest. systemd must reach running. Then tests/gate.sh boots the digest under systemd and runs the existing tests/smoke-test.sh inside it. Both pass against the currently published image.
  • Cascade: :latest keeps tracking :main. The repo is rebuilt on parent-image-updated and dispatches the same event to its children, as before.
  • Pull requests now build and gate. GHCR dropped. Inherits constitution v1.39.1.

Test plan

  • systemd gate + smoke test against the current published image
  • PR run: build, gates, mark green

🤖 Generated with Claude Code

https://claude.ai/code/session_01R7Y1jh5QhdxRuFKyWVev5h

build.yml becomes container.yml calling constitution's
[email protected]. The gates run against the pushed digest:
systemd must reach running, then tests/gate.sh runs the existing smoke
tests inside a fresh container of that digest (all pass against the
current published image). A cascade image: :latest keeps tracking :main,
and promotion dispatches parent-image-updated to the same children as
before. Pull requests now build and gate. GHCR is dropped; the repo
inherits constitution v1.39.1.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01R7Y1jh5QhdxRuFKyWVev5h

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gatehouse found 1 issues (1 high)

Full review.

2 finding(s) dropped: their evidence is not in the diff.


name: Container

on:

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HIGH (Constitution): The workflow has no weekly cron trigger, despite the project requirement that workflows include one for base-image security updates.

Suggestion: Add a weekly schedule trigger to this workflow.

Evidence:

on:
  pull_request:
  push:
    branches: [main]
    tags: ["v*"]
  repository_dispatch:
    types: [parent-image-updated]
  workflow_dispatch:

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not a bug: this is a cascade child. The weekly errata rebuild starts at the chain's root, ubi10-core (its schedule, 0 6 * * 1), whose promotion dispatches parent-image-updated down the chain, so this image is rebuilt weekly through its own gates. A cron of its own would rebuild against an un-refreshed parent. The Container Image profile's cascade section covers this, and this workflow had no cron before the PR either.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gatehouse found no issues.

Reviewed 1 commit since befe98f (incremental).

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gatehouse found no issues.

Reviewed 1 commit since ce39bc0 (incremental).

@fatherlinux
fatherlinux merged commit 82c06e0 into main Nov 11, 2026
11 checks passed
@fatherlinux
fatherlinux deleted the feat/container-release branch November 11, 2026 16:24
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