Skip to content

feat: Add native container build step for proxy (#124)#146

Open
Vaishnav88sk wants to merge 2 commits into
reshaprio:mainfrom
Vaishnav88sk:issue-124-native-proxy
Open

feat: Add native container build step for proxy (#124)#146
Vaishnav88sk wants to merge 2 commits into
reshaprio:mainfrom
Vaishnav88sk:issue-124-native-proxy

Conversation

@Vaishnav88sk

@Vaishnav88sk Vaishnav88sk commented May 24, 2026

Copy link
Copy Markdown
Contributor

fixes #124

@Vaishnav88sk Vaishnav88sk requested a review from lbroudoux as a code owner May 24, 2026 18:31
@Vaishnav88sk Vaishnav88sk force-pushed the issue-124-native-proxy branch from 0cf1de5 to c67c7c1 Compare May 24, 2026 18:33

@lbroudoux lbroudoux left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for pushing this. We need our container images to support multiple architectures (both amd64 and arm64). Please check my review comment.

Comment thread .github/workflows/build-containers.yml Outdated
@lbroudoux lbroudoux added question Further information is requested theme/build theme/install enhancement New feature or request labels May 26, 2026
@Vaishnav88sk Vaishnav88sk requested a review from lbroudoux May 27, 2026 08:51
Adds a new step to the GitHub Actions workflow to build a GraalVM native binary for the proxy module utilizing mandrel in a container, and publishes it securely using cosign.

Signed-off-by: Vaishnav88sk <vaishnavsk8804@gmail.com>
Extracts the native proxy build from the standard container build workflow and puts it into a dedicated, parallel workflow that uses matrix builds across native amd64 and arm64 runners to avoid slow QEMU emulation. It builds, pushes, and then stitches them together with a docker manifest.

Signed-off-by: Vaishnav88sk <vaishnavsk8804@gmail.com>
@Vaishnav88sk Vaishnav88sk force-pushed the issue-124-native-proxy branch from 9f0a485 to f3bc923 Compare June 8, 2026 14:49
@Vaishnav88sk

Copy link
Copy Markdown
Contributor Author

I removed the native proxy signing step from build-containers.yml during the rebase conflict resolution. Since the goal of this PR is to decouple the native builds, both the native proxy build and its signing step were moved entirely into the new build-native-containers.yml workflow!

@Vaishnav88sk

Copy link
Copy Markdown
Contributor Author

@lbroudoux I think it is good to merge now, but your review of code chnages is required to validate these things. Please take a look!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request question Further information is requested theme/build theme/install

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Package proxy component as a GraalVM native container image

2 participants