Skip to content

ci: add Docker build/push workflow to GHCR#109

Merged
mohit-nagaraj merged 1 commit into
mainfrom
ci/docker-build-push
Apr 14, 2026
Merged

ci: add Docker build/push workflow to GHCR#109
mohit-nagaraj merged 1 commit into
mainfrom
ci/docker-build-push

Conversation

@mohit-nagaraj

Copy link
Copy Markdown
Member

Summary

  • Add docker-publish.yml workflow that builds and pushes to ghcr.io/kubeorch/core on push to main (latest tag) and version tags (v*)
  • Multi-platform: linux/amd64 + linux/arm64
  • Fix Dockerfile: glob pattern for optional config.yaml copy (was failing when file didn't exist)
  • Bump Go version in Dockerfile from 1.22 to 1.25 to match go.mod

Test plan

  • Workflow triggers on merge to main
  • Image is pushed to ghcr.io/kubeorch/core:latest
  • Image runs correctly: docker run ghcr.io/kubeorch/core:latest

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the Dockerfile to use Go 1.25 and modifies the configuration file copy command to use a glob pattern for optional inclusion. However, several critical issues were identified: Go 1.25 is not a released version, which will cause build failures; the build process hardcodes the architecture despite claims of multi-platform support; and a referenced workflow file is missing from the PR. Additionally, the glob pattern for configuration files may inadvertently include unnecessary files like examples in the production image.

Comment thread Dockerfile
Comment thread Dockerfile Outdated
- Add docker-publish.yml: builds and pushes ghcr.io/kubeorch/core on
  push to main (latest) and version tags (v*); multi-platform
  linux/amd64 + linux/arm64 via TARGETARCH/TARGETOS build args
- Fix Dockerfile: bump Go 1.22 -> 1.23 (latest stable), use ARG
  TARGETOS/TARGETARCH for correct multi-platform binaries, remove
  config.yaml copy (configuration is provided via env vars or a
  mounted file at runtime)

Signed-off-by: Mohit Nagaraj <mohitnagaraj20@gmail.com>
@mohit-nagaraj
mohit-nagaraj merged commit 5561845 into main Apr 14, 2026
2 checks passed
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