Skip to content

ci: add Docker build-only job to CI - #9

Merged
Jonathan0823 merged 2 commits into
mainfrom
ci/docker-build
May 11, 2026
Merged

Jonathan0823 merged 2 commits into
mainfrom
ci/docker-build

Conversation

@Jonathan0823

@Jonathan0823 Jonathan0823 commented May 11, 2026 •

Copy link
Copy Markdown
Owner

Related issue

  • None

Summary

  • Added a Docker build check to CI alongside existing pytest job
  • Job runs docker build against the existing Dockerfile; no push
  • Catches broken Dockerfile syntax, missing system deps, and build-time failures early

Scope

  • In scope: Dockerfile build success in CI
  • Out of scope: docker-compose, multi-service tests, image pushes/registry

Testing

  • Tests run (pytest)
  • Manual Docker build (will run in CI)

Test details

  • Command: docker build .
  • Expected: exit code 0, no missing Alpine packages or build errors

Reviewer focus

  • Dockerfile correctness on Alpine
  • CI workflow YAML syntax and permissions

Notes

  • No migrations or config changes
  • Follow-ups: optional image build & push for deployments

Summary by CodeRabbit

  • Chores
    • Integrated Docker image building into the continuous integration workflow to automate container creation during the development and deployment pipeline.

Review Change Stack

- adds docker-build job using docker/build-push-action@v5
- builds image from Dockerfile but does not push
- runs on same triggers as test job (push/PR to main)
- fails fast if Dockerfile or dependency set is broken
@coderabbitai

coderabbitai Bot commented May 11, 2026 •

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

A new GitHub Actions CI job was added to build the Docker image defined in the repository's Dockerfile. The docker-build job runs on every push, uses Docker Buildx to construct the image from the repository root, tags it as nexusbus:dev, and does not push the result to a registry.

Changes

CI Docker Build Configuration

Layer / File(s) Summary
Docker Build Job
.github/workflows/ci.yml
New docker-build job added to CI workflow. Checks out repository, sets up Docker Buildx, builds Docker image from ./Dockerfile with context ., tags result as nexusbus:dev, and disables push (push: false).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A Docker job springs forth with grace,
Building images in the CI space,
No push to the registry today,
Just nexusbus:dev on its way! 📦

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'ci: add Docker build-only job to CI' directly and clearly summarizes the main change: adding a new Docker build job to the CI workflow, which aligns with the file changes in .github/workflows/ci.yml.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/docker-build

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.

- enables load=true to keep image locally after build
- adds inspect step to extract .Size from image metadata
- converts bytes to MB with 2 decimal precision
- writes size to GITHUB_STEP_SUMMARY for visibility
@Jonathan0823
Jonathan0823 merged commit 1d3d7fb into main May 11, 2026
4 checks passed
@Jonathan0823
Jonathan0823 deleted the ci/docker-build branch May 11, 2026 09:31
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