Skip to content
This repository has been archived by the owner on Nov 7, 2023. It is now read-only.

add docker buildx support #61

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

wawlian
Copy link
Contributor

@wawlian wawlian commented Apr 13, 2023

What type of PR is this?

/kind enhancement

What this PR does / why we need it:

add docker buildx multi-arch image building support

Which issue(s) this PR fixes:

none

Special notes for your reviewer:

@lwmqwer

Does this PR introduce a user-facing change?

users can now build multi-arch image with export IMG=xxx;make buildx-release

@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Merging #61 (2951f8f) into main (9b6aabe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #61   +/-   ##
=======================================
  Coverage   45.40%   45.40%           
=======================================
  Files          14       14           
  Lines        1731     1731           
=======================================
  Hits          786      786           
  Misses        839      839           
  Partials      106      106           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@LavenderQAQ LavenderQAQ left a comment

Choose a reason for hiding this comment

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

/lgtm

@openyurt-bot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: LavenderQAQ, wawlian
To complete the pull request process, please assign rambohe-ch
You can assign the PR to them by writing /assign @rambohe-ch in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@LavenderQAQ LavenderQAQ removed their assignment Apr 17, 2023
@openyurt-bot openyurt-bot removed the lgtm label Apr 18, 2023
@openyurt-bot
Copy link

New changes are detected. LGTM label has been removed.

@wawlian wawlian force-pushed the feature/docker-buildx branch 2 times, most recently from 879aead to 2178334 Compare April 18, 2023 07:59
@wawlian wawlian force-pushed the feature/docker-buildx branch 4 times, most recently from e7b1b49 to eaa06de Compare April 18, 2023 08:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants