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_v2: allow to pass --build or --no-build to 'docker compose up' #760

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

felixfontein
Copy link
Collaborator

SUMMARY

Ref: #750
Ref: #739 (comment)

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

docker_compose_v2

@felixfontein felixfontein added the docker-compose-v2 Docker Compose v2 label Jan 16, 2024
@felixfontein felixfontein changed the title Allow to pass --build or --no-build to 'docker compose up'. docker_compose_v2: allow to pass --build or --no-build to 'docker compose up' Jan 16, 2024
@felixfontein felixfontein merged commit 22d956e into ansible-collections:main Jan 17, 2024
122 of 123 checks passed
@felixfontein felixfontein deleted the build branch January 17, 2024 05:57
- Whether to build images before starting containers. This is used when O(state=present).
- V(always) always builds before starting containers. This is equivalent to the C(--build) option of C(docker compose up).
- V(never) never builds before starting containers. This is equivalent to the C(--no-build) option of C(docker compose up).
- V(policy) uses the policy as defined in the Copose file.

Choose a reason for hiding this comment

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

Thank you.

Here's a minor typo (Copose).

The description for the build option This is used when O(state=present). is different from the pull option: This is used when C(docker compose up) is ran.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm currently not sure which of the two descriptions is better. Probably the one for pull, since state=stopped can also use docker compose up --no-start to create the containers.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm adjusting it in #766.

@felixfontein
Copy link
Collaborator Author

@pgassmann @Qlas thanks, fixed I fixed the typo in c3322fd.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker-compose-v2 Docker Compose v2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants