Skip to content

Install v2.20.0 of docker-compose-plugin#6997

Merged
pgporada merged 4 commits into
mainfrom
fix-ci-docker-compose
Jul 13, 2023
Merged

Install v2.20.0 of docker-compose-plugin#6997
pgporada merged 4 commits into
mainfrom
fix-ci-docker-compose

Conversation

@aarongable

@aarongable aarongable commented Jul 13, 2023

Copy link
Copy Markdown
Contributor

Work around the bug[1] in docker-compose-plugin v2.19.1, which is included in the current github actions ubuntu 20.04 runner image[2].

[1] docker/compose#10777
[2] actions/runner-images#7916

Part of #6998

@aarongable aarongable marked this pull request as ready for review July 13, 2023 21:16
@aarongable aarongable requested a review from a team as a code owner July 13, 2023 21:16
@aarongable aarongable requested a review from jsha July 13, 2023 21:16

# TODO(#6998): Remove this step.
- name: Update docker compose plugin
run: mkdir -p ${DOCKER_CONFIG:-$HOME/.docker}/cli-plugins && curl -SL https://github.com/docker/compose/releases/download/v2.20.0/docker-compose-linux-x86_64 -o ${DOCKER_CONFIG:-$HOME/.docker}/cli-plugins/docker-compose && chmod +x ${DOCKER_CONFIG:-$HOME/.docker}/cli-plugins/docker-compose

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you link to some documentation on what the cli-plugins directory does? Also this looks like the command is copy-pasted from somewhere, which is fine, but we should link to it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

They're from https://docs.docker.com/compose/install/linux/#install-the-plugin-manually. Comment updated. The cli-plugins directory is just a place where the docker command looks for executables that it can expose as subcommands, much like git searches $PATH for executables named git-something and exposes them as git something subcommands.

@aarongable aarongable requested a review from jsha July 13, 2023 21:26
@pgporada pgporada merged commit 944a865 into main Jul 13, 2023
@pgporada pgporada deleted the fix-ci-docker-compose branch July 13, 2023 21:33
aarongable added a commit that referenced this pull request Aug 1, 2023
pgporada pushed a commit that referenced this pull request Aug 2, 2023
Reverts #6997, which manually
installed a specific version of the docker compose cli plugin to work
around a bug in the version installed by default. The images now ship
the fixed version by default
(actions/runner-images#7916 (comment)),
so we can remove the workaround.

Fixes #6998
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants