Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Publish multi-arch docker images and binaries for arm64, amd64 #3847

Merged
merged 1 commit into from
Dec 13, 2022

Conversation

jkneubuh
Copy link
Contributor

@jkneubuh jkneubuh commented Dec 12, 2022

Signed-off-by: Josh Kneubuhl jkneubuh@us.ibm.com

Type of change

  • New feature

Description

Builds multi-arch Docker images and native binaries for arm64 and amd64.

Additional details

This PR is a "cherry-pick" (back/forward) port of the following PRs from the release-2.5 branch "up" into the mainline:

Related issues

@jkneubuh jkneubuh requested a review from a team as a code owner December 12, 2022 18:44
@jkneubuh jkneubuh changed the title Publish multi-arch docker images and binaries for arm64, amd64 - DO NOT MERGE Publish multi-arch docker images and binaries for arm64, amd64 Dec 12, 2022
@denyeart
Copy link
Contributor

I think this looks good and is an accurate forward port of the release-2.5 changes that have been validated in that branch.

The changes to the Dockerfiles are slightly different than what was done in release-2.5, because Artem @C0rWin has previously updated the Dockerfiles for ARM in ac31c57. So let's ask @C0rWin to review also and make sure he is good with this approach.

@denyeart
Copy link
Contributor

The etcdraft unit test failure is a known flake and can be ignored, it is being looked at by @Param-S

@denyeart
Copy link
Contributor

I'll go ahead and merge to keep the branches consistent, but @C0rWin and others please do holler if you see anything not correct.

Signed-off-by: Josh Kneubuhl <jkneubuh@us.ibm.com>
@denyeart denyeart merged commit cfeb7f7 into hyperledger:main Dec 13, 2022
@denyeart
Copy link
Contributor

Note that subsequent PR #3857 reversed some of this.

As of #3857, the locally built peer images will continue to be created as dynamically linked binaries, with the static linking enabled only for the release target and for binaries deployed to alpine in Docker.

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