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

Fixes #398: exclude deps on up if --no-deps #1108

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bailsman
Copy link

Fixes #398 by excluding dependent services if --no-deps is given on the command line.

@p12tic
Copy link
Collaborator

p12tic commented Jan 14, 2025

Thanks for the PR. It looks good.

Please a release note to the newsfragments directory (you can look here for inspiration on how release note looks like).

Also the PR needs an integration test. In this particular case I think tests/integration/test_podman_compose_deps.py is most relevant for enhancement. Note that it uses compose files from tests/integration/deps.

@bailsman bailsman force-pushed the fix-398 branch 3 times, most recently from 632f377 to 088ff46 Compare January 15, 2025 06:46
@bailsman
Copy link
Author

bailsman commented Jan 15, 2025

Thanks for the hints! The integration test exposed that it is a bit more involved than I thought initially (my local setup didn't trigger these).

  • Run (through container_to_args) adds --requires parameters to the podman run command line, which does not work if the dependency is not created/started. Worked around this by adding extra boolean parameter to container_to_args, as is already done for detach.
  • Up tries to check if dependencies have successfully started, worked around this by passing in an empty deps set if no-deps is given.

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.

option "--no-deps" is ignored for "podman-compose up"
2 participants