Skip to content

chore(build) Updating dockerfile(s) with buildx built-in ARGs#503

Merged
akhilerm merged 1 commit intomasterfrom
feature/dockerfile
Oct 27, 2020
Merged

chore(build) Updating dockerfile(s) with buildx built-in ARGs#503
akhilerm merged 1 commit intomasterfrom
feature/dockerfile

Conversation

@xunholy
Copy link
Copy Markdown
Contributor

@xunholy xunholy commented Oct 18, 2020

Signed-off-by: Michael Fornaro 20387402+xUnholy@users.noreply.github.com

Updating Dockerfiles to use the buildx built-in ARGs will provide more flexibility to provide support for additional OS/ARCHS/ARM variants in the future. This also will create a cleaner and more maintainable Dockerfile moving forward.

https://docs.docker.com/engine/reference/builder/#automatic-platform-args-in-the-global-scope

Checklist:

  • Fixes #
  • PR Title follows the convention of <type>(<scope>): <subject>
  • Has the change log section been updated?
  • Commit has unit tests
  • Commit has integration tests
  • (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
  • (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them

Signed-off-by: Michael Fornaro <20387402+xUnholy@users.noreply.github.com>

update release changelog

Signed-off-by: Michael Fornaro <20387402+xUnholy@users.noreply.github.com>
@xunholy xunholy force-pushed the feature/dockerfile branch from 085ab53 to 9d1537b Compare October 18, 2020 03:55
Comment thread build/ndm-daemonset/Dockerfile
Copy link
Copy Markdown
Contributor

@akhilerm akhilerm left a comment

Choose a reason for hiding this comment

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

@xunholy : given a comment / question.

Copy link
Copy Markdown
Contributor

@akhilerm akhilerm left a comment

Choose a reason for hiding this comment

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

/lgtm

@akhilerm akhilerm requested a review from kmova October 18, 2020 12:35
@akhilerm akhilerm merged commit 136bc4d into master Oct 27, 2020
@akhilerm akhilerm deleted the feature/dockerfile branch October 27, 2020 05:05
@prateekpandey14 prateekpandey14 added the Hacktoberfest Issue / PR is part of Hacktoberfest label Oct 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Hacktoberfest Issue / PR is part of Hacktoberfest hacktoberfest-accepted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants