Skip to content

chore: implement make target for multi arch image build#139

Merged
arkodg merged 5 commits into
envoyproxy:mainfrom
daixiang0:multibuild
Jul 8, 2022
Merged

chore: implement make target for multi arch image build#139
arkodg merged 5 commits into
envoyproxy:mainfrom
daixiang0:multibuild

Conversation

@daixiang0
Copy link
Copy Markdown
Member

Fix #138

Signed-off-by: Loong Dai loong.dai@intel.com

Signed-off-by: Loong Dai <loong.dai@intel.com>
@daixiang0 daixiang0 requested a review from a team as a code owner July 6, 2022 02:15
Signed-off-by: Loong Dai <loong.dai@intel.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #139 (5779bbf) into main (2429e81) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #139   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            5         5           
=========================================
  Hits             5         5           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2429e81...5779bbf. Read the comment docs.

Comment thread tools/make/image.mk Outdated
Comment thread tools/make/image.mk
daixiang0 added 3 commits July 7, 2022 09:13
Signed-off-by: Loong Dai <loong.dai@intel.com>
Signed-off-by: Loong Dai <loong.dai@intel.com>
Signed-off-by: Loong Dai <loong.dai@intel.com>
Copy link
Copy Markdown
Contributor

@arkodg arkodg left a comment

Choose a reason for hiding this comment

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

LGTM thanks !

Sidenote - would have been great if --load worked with buildx/docker to load the manifest list into docker but it doesnt
docker/buildx#59

@arkodg arkodg merged commit e945a38 into envoyproxy:main Jul 8, 2022
@daixiang0 daixiang0 deleted the multibuild branch July 8, 2022 00:39
Comment thread tools/make/image.mk

.PHONY: image.multiarch.setup
image.multiarch.setup: image.verify image.multiarch.verify image.multiarch.emulate
@docker run --rm --privileged tonistiigi/binfmt --install all # Install QEMU emulators
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

shouldnt this line be deleted @daixiang0 ?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I miss it, post anther PR to fix it.

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.

Implement make target for multi arch image build

3 participants