Skip to content

commands/.../cmd: pass along Docker Build Args#1034

Merged
theishshah merged 6 commits intooperator-framework:masterfrom
theishshah:docker-build-args
Feb 1, 2019
Merged

commands/.../cmd: pass along Docker Build Args#1034
theishshah merged 6 commits intooperator-framework:masterfrom
theishshah:docker-build-args

Conversation

@theishshah
Copy link
Contributor

Description of the change:
Adds --docker-build-args flag to the operator-sdk build command.

Motivation for the change:
Closes #846

@theishshah theishshah requested a review from AlexNPavel January 30, 2019 22:01
@openshift-ci-robot openshift-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 30, 2019
@openshift-ci-robot openshift-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jan 31, 2019
Copy link
Contributor

@AlexNPavel AlexNPavel left a comment

Choose a reason for hiding this comment

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

LGTM

@openshift-ci-robot openshift-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jan 31, 2019
@theishshah theishshah changed the title Pass along Docker Build Args commands/.../cmd: pass along Docker Build Args Jan 31, 2019
Copy link
Member

@joelanford joelanford left a comment

Choose a reason for hiding this comment

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

LGTM other than a couple of nits on the help text.

@theishshah theishshah merged commit 90b6980 into operator-framework:master Feb 1, 2019
@theishshah theishshah deleted the docker-build-args branch February 1, 2019 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add possibility to declare proxy as build-arg to docker build

4 participants