Skip to content

Add multi-arch image build and push to Quay.io#4

Merged
clcollins merged 3 commits into
claude/add-copilot-evaluation-R8OiQfrom
chcollin/image-build-push
Mar 31, 2026
Merged

Add multi-arch image build and push to Quay.io#4
clcollins merged 3 commits into
claude/add-copilot-evaluation-R8OiQfrom
chcollin/image-build-push

Conversation

@clcollins

@clcollins clcollins commented Mar 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add image-build-push.yaml workflow for multi-arch (amd64+arm64) container image builds pushed to quay.io/chcollin/gort
  • Use Quay robot account secrets (QUAY_ROBOT_USERNAME/QUAY_ROBOT_TOKEN) matching dwarfbot PR docs: add setup guide for integrations #11
  • Update Containerfile with OCI labels and remove hardcoded GOARCH for multi-platform support
  • Update CI image-build job to validate OCI labels with podman

Test plan

  • Verify QUAY_ROBOT_USERNAME and QUAY_ROBOT_TOKEN secrets are set in repo settings
  • Confirm PR build triggers image-build-push workflow (build only, no push)
  • Merge and confirm push to main triggers a successful image build and push to quay.io/chcollin/gort:latest
  • Tag a release and confirm multi-tag push works (sha, v-prefixed, bare version)

🤖 Generated with Claude Code

Add image-build-push workflow using Quay robot account secrets
(QUAY_ROBOT_USERNAME/QUAY_ROBOT_TOKEN), multi-arch builds (amd64+arm64),
and standard tagging (SHA, pr-N, version, latest). Update Containerfile
with OCI labels and remove hardcoded GOARCH for multi-platform support.
Update CI image-build job with OCI label validation.

Based on clcollins/dwarfbot image-build-push workflow with PR #11 changes.

Created with assistance from Claude 🤖 <claude@anthropic.com>

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Christopher Collins <collins.christopher@gmail.com>

Copilot AI 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.

Pull request overview

Adds automated multi-arch (amd64/arm64) container image build & publish to Quay.io, and standardizes image metadata via OCI labels so the CI pipeline can validate built artifacts.

Changes:

  • Add OCI/metadata labels to the runtime image and remove the hardcoded GOARCH to support multi-platform builds.
  • Introduce a new GitHub Actions workflow to build amd64+arm64 images and push a multi-arch manifest to Quay on main/tag events.
  • Update existing CI container build job to use Podman and validate required OCI labels via podman inspect.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
Containerfile Enables multi-arch builds and adds OCI + project-specific labels via build args.
.github/workflows/image-build-push.yaml New workflow to build amd64/arm64 images, create a manifest list, and push tags to Quay (skips push on PRs).
.github/workflows/ci.yaml Switches CI image build to Podman and adds label validation checks.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Containerfile Outdated
Comment thread .github/workflows/image-build-push.yaml Outdated
Remove quotes from Containerfile ARG defaults to prevent literal
quote characters in label values. Update PR build log messages
to accurately reflect that only the SHA manifest is created locally.

Created with assistance from Claude 🤖 <claude@anthropic.com>

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Christopher Collins <collins.christopher@gmail.com>

Copilot AI 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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/image-build-push.yaml Outdated
Comment thread .github/workflows/ci.yaml Outdated
Use quay.io/chcollin/gort in both workflows to match the Makefile
IMAGE_REGISTRY default, avoiding mismatch between CI and local builds.

Created with assistance from Claude 🤖 <claude@anthropic.com>

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Christopher Collins <collins.christopher@gmail.com>

Copilot AI 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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/image-build-push.yaml
Comment thread .github/workflows/image-build-push.yaml

Copilot AI 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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@clcollins

Copy link
Copy Markdown
Owner Author

@clcollins PR #4 has been fully qualified.

Qualification Summary:

  • Loops: 3 iteration(s) of Phases 1–4
  • New commits: 2 commit(s) added during qualification
  • CI: ⏭️ N/A (no checks configured for claude/add-copilot-evaluation-R8OiQ base branch)
  • Copilot: ✅ Clean after 4 rounds — 4 comments addressed (ARG quotes, log messages, registry alignment, PR description)
  • CodeRabbit: ⏭️ N/A (not enabled)
  • Codecov: ⏭️ N/A (no checks on this base branch)
  • Merge: ✅ No conflicts (MERGEABLE/CLEAN)

Intent validation: The PR still fulfills its original intent. All changes made during qualification (Copilot feedback on ARG quoting, log message accuracy, and registry name consistency) are consistent with the PR's purpose of adding multi-arch image build and push to Quay.io.

This PR is ready to merge.


Qualified by Claude on behalf of @clcollins

@clcollins clcollins merged commit 4c7d59f into claude/add-copilot-evaluation-R8OiQ Mar 31, 2026
8 checks passed
@clcollins clcollins mentioned this pull request Mar 31, 2026
10 tasks
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.

2 participants