-
Notifications
You must be signed in to change notification settings - Fork 150
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
Bump compose v2.33.0 #1163
Bump compose v2.33.0 #1163
Conversation
Signed-off-by: Nicolas De Loof <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -104,6 +104,7 @@ Homepage: https://github.com/docker/buildx | |||
Package: docker-compose-plugin | |||
Priority: optional | |||
Architecture: linux-any | |||
Recommends: docker-buildx-plugin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Silly question; if compose shells out to buildx, and running as “standalone” (docker-compose
) does it shell out to /whatever/path/for /plugins/docker-buildx
(standalone) or docker buildx
? Mostly looking at Recommends:
in the docker-compose-plugin
package, and if that (when installed separate from the CLI) would also be able to use it, or if it should depend on docker-ce-cli
already having the Recommends:
for buildx
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
compose executes the CLI plugin directly (standalone) based on binary file path retrieved from manager.GetPlugin
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gotcha; thanks! Wanted to double-check we wouldn't end up in a case where the plug-in was there, but compose still trying to execute it in a different location.
We should probably look if we can have good ways for the CLI (when compose is executed as plugin) to pass information to the plugin(s), so that we have a single source of truth for resolving plugins, but that's an existing problem.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- What I did
Bump compose v2.33.0
Declare buildx as a recommended dependency as v2.33.0 introduces support for delegating build command to bake.
- How I did it
- How to verify it
- Description for the changelog
Compose v2.33.0
- A picture of a cute animal (not mandatory but encouraged)