Skip to content

feat(ci): periodic overlay-image-pull smoke test#191

Merged
cbeaulieu-gt merged 1 commit into
mainfrom
issue-190-overlay-smoke-test
May 5, 2026
Merged

feat(ci): periodic overlay-image-pull smoke test#191
cbeaulieu-gt merged 1 commit into
mainfrom
issue-190-overlay-smoke-test

Conversation

@cbeaulieu-gt
Copy link
Copy Markdown
Member

Summary

  • Adds .github/workflows/overlay-smoke.yml — runs daily at 06:17 UTC and on workflow_dispatch; attempts docker pull for each of the three digest-pinned overlay images (review, fix, explain)
  • Digests are grepped at runtime from the existing claude-*.yml workflow files — no separate list to drift
  • On failure: opens a deduped GitHub issue titled Overlay smoke test failed: <verb> (or appends a comment to the existing open issue); workflow exits non-zero so the run shows as failed
  • Documents the smoke workflow in CLAUDE.md § CI Runtime (Phase 1+)

Test plan

  • Push triggers actionlint lint check — should pass (verified locally: zero errors)
  • Manually trigger overlay-smoke workflow via workflow_dispatch to confirm all three pulls succeed on current digests
  • Verify matrix shows three parallel jobs: review, fix, explain
  • To test failure path: temporarily edit a digest pin to an invalid SHA, trigger manually, confirm an issue is opened titled Overlay smoke test failed: <verb>

Closes #190

🤖 Generated by Claude Code on behalf of @cbeaulieu-gt

@cbeaulieu-gt cbeaulieu-gt force-pushed the issue-190-overlay-smoke-test branch from d839c1a to bd2050f Compare May 5, 2026 19:29
cbeaulieu-gt added a commit that referenced this pull request May 5, 2026
Phase 2 base image was missing two binaries that consumer composite
actions hard-depend on:

- `unzip` is required by oven-sh/setup-bun@v1, used during the setup
  phase of anthropics/claude-code-action@v1. Without it the action
  fails with "Unable to locate executable file: unzip" before the
  review body even runs (observed on PR #191 after #193 merged).

- `gh` is invoked at 14+ call sites across pr-review/, apply-fix/,
  lint-failure/, ci-failure/, and tag-claude/ composite actions
  (PR diff fetching, status creation, comment posting, dedup). The
  AI itself shells out to gh via --allowedTools.

Refs #194. PR-B will follow with digest-pin updates in the 5
container-pinned reusable workflows once the post-merge
runtime-build run on main produces the new digests.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds .github/workflows/overlay-smoke.yml — a scheduled (daily 06:17 UTC)
plus workflow_dispatch workflow that attempts docker pull for each of the
three digest-pinned overlay images (review, fix, explain).

Digest source of truth: the existing claude-*.yml workflow files, grepped
at runtime, so no separate digest list that could drift.  Failure path
deduplicates GitHub issues by stable title so repeated failures append a
comment rather than opening a new issue.  actionlint passes cleanly.

Also documents the smoke workflow in CLAUDE.md § CI Runtime (Phase 1+).

closes #190

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cbeaulieu-gt cbeaulieu-gt force-pushed the issue-190-overlay-smoke-test branch from bd2050f to 9ff4871 Compare May 5, 2026 20:02
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Claude Code is working…

I'll analyze this and get back to you.

View job run

@cbeaulieu-gt cbeaulieu-gt merged commit 30fe294 into main May 5, 2026
2 of 3 checks passed
@cbeaulieu-gt cbeaulieu-gt deleted the issue-190-overlay-smoke-test branch May 5, 2026 20:13
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.

Add periodic smoke test for overlay image pulls

1 participant