Skip to content

fix(image): capture collector:user's output in the smoke test, and roadmap notes - #89

Merged
plainprogrammer merged 5 commits into
mainfrom
031-image-vendor-and-smoke-check
Oct 14, 2026
Merged

plainprogrammer merged 5 commits into
mainfrom
031-image-vendor-and-smoke-check

Conversation

@plainprogrammer

Copy link
Copy Markdown
Owner

Three touch-ups from the review of PR #82 that were committed after it merged, plus the roadmap notes that PR couldn't carry.

What changed

Verification

Notes

🤖 Generated with Claude Code

plainprogrammer and others added 5 commits October 13, 2026 21:37
…smoke test (#19)

The collector:user check still piped the task into grep -q, the shape #19 names: under pipefail, a grep that
ends at its first match can fail the pipeline. It now captures the output, fails with it if the task exits
non-zero, then asserts the whole line "Created admin smoke@example.com." and shows what it got when the line
is missing. No check in the script pipes into grep -q now.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The check rejects anything in /rails/vendor besides .keep, javascript and ocr; it does not require them, so an
empty listing passes. Its heading said "holds only". It now says "nothing unexpected", and a comment says
that the OCR files served above are what show ocr is there.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e the images ran (#19, #31)

The failure paths were shown with a stand-in container CLI, and every image run was podman on amd64. The
follow-ups memory points at PR #82.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ekeeping lane

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@plainprogrammer plainprogrammer self-assigned this Oct 14, 2026
@plainprogrammer
plainprogrammer merged commit a821389 into main Oct 14, 2026
4 checks passed
@plainprogrammer
plainprogrammer deleted the 031-image-vendor-and-smoke-check branch October 14, 2026 02:11
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.

Tighten bin/image-smoke's catalog:status check

1 participant