Skip to content

chore: bump container images to sha-18f19e1 / sha-0385cea#167

Merged
DavidCozens merged 1 commit into
mainfrom
chore/bump-container-image-libssl
Apr 20, 2026
Merged

chore: bump container images to sha-18f19e1 / sha-0385cea#167
DavidCozens merged 1 commit into
mainfrom
chore/bump-container-image-libssl

Conversation

@DavidCozens

@DavidCozens DavidCozens commented Apr 20, 2026

Copy link
Copy Markdown
Owner

Purpose

Prerequisite for E3 TLS transport (first story: S3.7). The new `SolidSyslogPosixTlsStream` component will link against libssl/libcrypto, so both compiler containers now ship `libssl-dev` (headers + linker symlinks).

Change Description

  • `cpputest`: `sha-6acaccf` → `sha-18f19e1` (brings in libssl-dev via CppUTestDocker#1)
  • `cpputest-clang`: `sha-6ea3f95` → `sha-0385cea` (brings in libssl-dev via CppUTestDockerClang#1)

Also corrects drift in `docs/containers.md` — it had `sha-e7aa8a1` listed for `cpputest` while `.devcontainer/docker-compose.yml` and `.github/workflows/ci.yml` actually used `sha-6acaccf`.

No Windows / BDD-Windows / Behave container references changed — only the Linux compiler-container jobs (`build-and-test`, `clang-build-and-test`, `sanitize`, `coverage`, `tidy`, `format`, `cppcheck`) and the two devcontainer compose services.

Test Evidence

After recreating the `gcc` and `clang` compose services on the new images, the full local preset gauntlet is green:

  • Verified OpenSSL dev files present in both new images (`libssl-dev 3.0.19-1~deb12u2`, `/usr/include/openssl/ssl.h`, `/usr/lib/x86_64-linux-gnu/libssl.so`)
  • `debug` — 582 tests passed
  • `clang-debug` — 582 tests passed
  • `sanitize` (ASan + UBSan) — 582 tests passed
  • `coverage` — 99.9% overall (1108/1109 lines), 100% functions
  • `tidy` (clang-tidy, all warnings-as-errors) — clean
  • `cppcheck` — clean
  • `format` (clang-format dry-run with `-Werror`) — clean
  • `bdd` (behave + syslog-ng) — 15 features / 32 scenarios / 168 steps passed

Areas Affected

  • Devcontainer — `gcc` and `clang` services. A "Dev Containers: Rebuild Container" is required locally on merge.
  • CI — all seven Linux container jobs run against the new images.
  • `docs/containers.md` — image-tag table brought back in sync.
  • No runtime behaviour change; no production code touched.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores

    • Updated container images for development and continuous integration environments to newer pinned versions.
  • Documentation

    • Updated container image references in documentation to reflect the latest versions.

- cpputest: sha-6acaccf → sha-18f19e1 (adds libssl-dev)
- cpputest-clang: sha-6ea3f95 → sha-0385cea (adds libssl-dev)

Prerequisite for E3 (TLS transport, #5) — SolidSyslogPosixTlsStream
links libssl/libcrypto, so the compiler containers need OpenSSL
development headers.

Also corrects the drift in docs/containers.md, which had sha-e7aa8a1
listed for cpputest while the compose and workflow files used
sha-6acaccf.

Verified locally on the new images: debug, clang-debug, sanitize,
coverage (99.9%, SolidSyslogStreamSender.c 100%), tidy, cppcheck,
format, and full BDD suite (15 features / 32 scenarios / 168 steps)
all green.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7bec2165-bc7c-48d6-9bd5-079e8b280520

📥 Commits

Reviewing files that changed from the base of the PR and between d1c04b3 and 1419401.

📒 Files selected for processing (3)
  • .devcontainer/docker-compose.yml
  • .github/workflows/ci.yml
  • docs/containers.md

📝 Walkthrough

Walkthrough

This pull request updates pinned container image SHAs for the cpputest and cpputest-clang GitHub Container Registry images across the dev container, CI workflows, and documentation. The GCC image SHA is updated to sha-18f19e1 and the Clang image SHA to sha-0385cea.

Changes

Cohort / File(s) Summary
Container Image Updates
.devcontainer/docker-compose.yml, .github/workflows/ci.yml, docs/containers.md
Updated pinned container image SHAs: cpputest from sha-6acaccf to sha-18f19e1 and cpputest-clang from sha-6ea3f95 to sha-0385cea.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A rabbit hops through registries bright,
New SHAs pinned, containers tight!
From old to new, the images dance,
With gcc and clang in perfect stance. 🐰✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change—bumping container images to specific SHA values—and is concise and specific.
Description check ✅ Passed The description covers all required sections: Purpose (links to prerequisite story), Change Description (specific image updates and rationale), Test Evidence (comprehensive testing), and Areas Affected (clear scope).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-container-image-libssl

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

Copy link
Copy Markdown
Contributor

☀️   Quality Summary

   🚦   Unit Tests (GCC): 99% successful (✔️ 589 passed, 🙈 3 skipped)
   🚦   Unit Tests (Clang): 99% successful (✔️ 543 passed, 🙈 3 skipped)
   🚦   Unit Tests (Sanitize): 99% successful (✔️ 543 passed, 🙈 3 skipped)
   🚦   BDD Tests (Linux): 100% successful (✔️ 32 passed)
   🚦   BDD Tests (Windows): 66% successful (✔️ 21 passed, 🙈 11 skipped)
   🚦   Unit Tests (MSVC): 100% successful (✔️ 433 passed, 🙈 1 skipped)
   ⚠️   Clang-Tidy: No warnings
   ⚠️   CPPCheck: No warnings


Created by Quality Monitor v1.14.0 (#f3859fd). More details are shown in the GitHub Checks Result.

@DavidCozens DavidCozens merged commit d6e91d3 into main Apr 20, 2026
13 checks passed
@DavidCozens DavidCozens deleted the chore/bump-container-image-libssl branch April 20, 2026 16:25
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