-
Notifications
You must be signed in to change notification settings - Fork 104
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
Update awsx to use docker-build #1349
Comments
I'd like to propose a different option here. Instead of adding the different docker resources, why don't we add a function/resource method to create the provider/auth data instead? The existing |
Yes I think this is a great idea. This gives users much more composable blocks to build from. Option 1: Include the credentials as an output of the registry resource in awsx - an object containing Option 2: Create an invoke which returns the same properties. This could live in either aws or awsx. Perhaps we do both? |
This sounds like a great idea! |
@flostadler to clarify, we aren't adding any new resources only changing the internals of what's currently available. The legacy docker provider is showing its age, and it would be good to prevent issues like e.g. pulumi/pulumi-docker#967 (comment) from biting the user. I agree that auth helpers would also be a great cohesive addition. |
Any ideas on when this make it out into production? We'd like to be able to use docker this way instead of having to make config changes |
docker-build is build is now the recommended way to build docker images, so we need to update the AWSX components to use it.
The text was updated successfully, but these errors were encountered: