Skip to content

Enabling '--secret' option in 'docker build' #4659

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

Closed

Conversation

h-michael
Copy link
Contributor

@h-michael h-michael commented Aug 14, 2020

This is a feature addition related to issue #2273.

Related: #2273

Description
This is PR for the addition of a feature that allows us to use the docker build --secret option.

Docker document link

Should I make sure to check the useBuildkit option?

This is a feature addition related to issue GoogleContainerTools#2273.
@tejal29
Copy link
Contributor

tejal29 commented Aug 28, 2020

@nkubala You were working on this right. Are you planning to use this approach or use existing --build-args

Copy link
Contributor

@tejal29 tejal29 left a comment

Choose a reason for hiding this comment

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

i think this is duplicate of #4731

@nkubala
Copy link
Contributor

nkubala commented Sep 11, 2020

@h-michael I realized you had submitted this as I was finishing up my PR last week, sorry! I had meant to get this earlier and then point you to it as a way to model your other PR you have open. let me get mine merged, and then if you can get your other PR to look similar to mine (config change + docs + tests), we can merge it quickly.

@h-michael
Copy link
Contributor Author

@nkubala Should I close this PR and fix #4660 ?

@nkubala nkubala self-assigned this Sep 17, 2020
@nkubala
Copy link
Contributor

nkubala commented Sep 18, 2020

@h-michael yes please! i'm gonna merge #4731 probably today, if you get #4660 to look like this one we can get it merged quickly

@h-michael h-michael closed this Sep 21, 2020
@h-michael h-michael deleted the docker-secrets-step1 branch September 21, 2020 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants