Skip to content

feat: add arm container support for harvest#4289

Closed
rahulguptajss wants to merge 2 commits into
mainfrom
rg2-buildx
Closed

feat: add arm container support for harvest#4289
rahulguptajss wants to merge 2 commits into
mainfrom
rg2-buildx

Conversation

@rahulguptajss
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 15, 2026 10:04
@cla-bot cla-bot Bot added the cla-signed label May 15, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Jenkins container build/publish pipeline and the Harvest Dockerfile to support building and publishing multi-architecture (amd64/arm64) container images.

Changes:

  • Replace legacy docker build/docker push flow with docker buildx build and multi-arch pushes (amd64 + arm64) in Jenkins.
  • Add a Buildx multi-arch builder setup stage (QEMU + buildx builder).
  • Update the container Dockerfile to cross-compile the Harvest binaries using TARGETARCH while building on BUILDPLATFORM.

Reviewed changes

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

File Description
jenkins/artifacts/jenkinsfile Switch pipeline to Buildx multi-arch build/push and update login/tagging behavior.
container/onePollerPerContainer/Dockerfile Enable multi-arch builds by compiling with GOARCH=$TARGETARCH and using BUILDPLATFORM for the builder stage.

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

Comment thread jenkins/artifacts/jenkinsfile
Comment thread jenkins/artifacts/jenkinsfile
Comment thread jenkins/artifacts/jenkinsfile
Comment thread jenkins/artifacts/jenkinsfile
Comment thread jenkins/artifacts/jenkinsfile Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants