Skip to content

Add image args for build linux arm64#830

Merged
Technoboy- merged 3 commits intomasterfrom
fix-1
Mar 11, 2026
Merged

Add image args for build linux arm64#830
Technoboy- merged 3 commits intomasterfrom
fix-1

Conversation

@Technoboy-
Copy link
Contributor

Add image args for build linux arm64

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@Technoboy- Technoboy- requested review from a team, freeznet and nlu90 as code owners March 11, 2026 02:56
@Technoboy- Technoboy- self-assigned this Mar 11, 2026
@github-actions github-actions bot added the no-need-doc This pr does not need any document label Mar 11, 2026
@freeznet freeznet requested a review from Copilot March 11, 2026 03:00
Copy link

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 introduces a configurable base image build argument for the Pulsar functions runner images, to support alternative base images (e.g., for linux/arm64 builds).

Changes:

  • Add a BASE_IMAGE build arg to the Java and Python runner Dockerfiles and use it for the final stage FROM.
  • Add a BASE_IMAGE build arg to the Go runner Dockerfile and switch FROM to reference it.

Reviewed changes

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

File Description
images/pulsar-functions-python-runner/Dockerfile Adds BASE_IMAGE arg and uses it for the runner stage base image.
images/pulsar-functions-java-runner/Dockerfile Adds BASE_IMAGE arg and uses it for the runner stage base image.
images/pulsar-functions-go-runner/Dockerfile Adds BASE_IMAGE arg and uses it for the (single) base image.

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

Copy link

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

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.

@Technoboy- Technoboy- merged commit 6d45977 into master Mar 11, 2026
14 of 16 checks passed
@Technoboy- Technoboy- deleted the fix-1 branch March 11, 2026 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-need-doc This pr does not need any document

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants