-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add support for --secret
from buildx
#539
Comments
Thank you for filing this feature request! |
EDIT: The comment below is outdated. We can securely add secrets to build args with version 4.4.2. It's not ideal though so supporting the Would like to add that doing something like this results in an error, so I don't think it's even possible to use v4 of this package to securely build images that require secrets. Or can someone point out a workaround?
|
This seems like a pretty big caveat. |
I already migrated to pulumi-docker 4 and realised I now can't use |
@mike-chen-samsung can you please explain your workaround and its limitations? Is there any chance of this being worked on in 2024? Currently:
|
@ShedPlant my workaround is the same as what you're doing - we must copy the creds into the docker image which invalidates cache |
Just adding a +1 that my team would also benefit from this feature. It is blocking us from using |
Hi folks, we just released a new Docker Build provider focused exclusively on building images with buildx/BuildKit. It provides The repository is here; our blog announcement is here; and finally API docs are here along with examples of how to migrate your existing We won't be able to add this functionality in this provider for a number of reasons, so I'm closing this as wont-fix. |
Very exciting, thanks @blampe ! |
Hello!
Issue details
Now that the
extraOptions
field is gone, there is no way of mounting a secret into the docker build, which is the recommended way of making use of secret keys when building.https://docs.docker.com/engine/reference/commandline/buildx_build/#secret
I imagine supporting the full
extraOptions
is infeasible because of the move away from the docker CLI, but it would be helpful to have this available as a top-level field.Affected area/feature
The text was updated successfully, but these errors were encountered: