Skip to content
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

docker_compose - Fix startup errors for docker-compose versions <1.17.0 #182

Merged

Conversation

Ajpantuso
Copy link
Collaborator

SUMMARY

Fixes docker_compose failures when starting services for docker-compose <1.17.0
Fixes #180

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

plugins/modules/docker_compose.py

ADDITIONAL INFORMATION
  • Only local testing was done for this using docker-compose ==1.16.0 vs docker-compose==1.29.0

@Ajpantuso Ajpantuso changed the title [WIP] docker_compose - Fix startup errors for docker-compose versions <1.17.0 docker_compose - Fix startup errors for docker-compose versions <1.17.0 Jul 27, 2021
@Ajpantuso Ajpantuso marked this pull request as ready for review July 27, 2021 22:37
@@ -146,6 +146,7 @@
description:
- Use with I(state) C(present) to stop all containers defined in the Compose file.
- If I(services) is defined, only the containers listed there will be stopped.
- Requires C(docker-compose) version 1.17.0 or greater.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- Requires C(docker-compose) version 1.17.0 or greater.
- Requires C(docker-compose) version 1.17.0 or greater for full support.
For older versions, the services will first be started and then stopped when
the service is supposed to be created as stopped.

@felixfontein felixfontein merged commit 03f649f into ansible-collections:main Jul 31, 2021
@felixfontein
Copy link
Collaborator

@Ajpantuso thanks for fixing this!

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.

docker_compose: Error starting project up() got an unexpected keyword argument 'start'
2 participants