Skip to content

CI: Add Docker image size testing.#161

Open
brycelelbach wants to merge 1 commit intomainfrom
feature/test-docker-image-sizes
Open

CI: Add Docker image size testing.#161
brycelelbach wants to merge 1 commit intomainfrom
feature/test-docker-image-sizes

Conversation

@brycelelbach
Copy link
Collaborator

@brycelelbach brycelelbach commented Mar 11, 2026

Summary

  • Add brev/test-docker-image-sizes.bash script to query compressed image sizes from GHCR and compare them against a baseline. Supports --tutorial all to test every tutorial, compare and history modes, and text/markdown/json output formats.
  • Add test-docker-image-sizes.yml workflow that runs after the build workflow completes, produces a summary table in the GitHub Actions job summary, and saves results as an artifact for PR commenting.
  • Add pr-comment-docker-image-sizes.yml workflow (triggered separately for security) that posts a PR comment when any tutorial image changes by more than 5%, following the established two-workflow pattern to avoid the Pwn Request vulnerability.

@brycelelbach brycelelbach force-pushed the feature/test-docker-image-sizes branch 2 times, most recently from 25b471e to a40ef35 Compare March 12, 2026 11:48
@brycelelbach brycelelbach changed the title CI: Add Docker image size testing and normalize workflow names. CI: Add Docker image size testing. Mar 12, 2026
Add test-docker-image-sizes.bash to query compressed image sizes from
GHCR and compare them against a baseline. Supports single tutorial,
all tutorials, and history modes with text, markdown, and JSON output.

Add test-docker-image-sizes.yml workflow triggered after builds to
produce a job summary table, and pr-comment-docker-image-sizes.yml to
warn on PRs when any image changes by more than 5%.
@brycelelbach brycelelbach force-pushed the feature/test-docker-image-sizes branch from a40ef35 to ff2873d Compare March 12, 2026 13: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.

1 participant