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

[BUG] docker compose up --no-deps recreates container #12069

Open
RicardoM17 opened this issue Aug 20, 2024 · 2 comments
Open

[BUG] docker compose up --no-deps recreates container #12069

RicardoM17 opened this issue Aug 20, 2024 · 2 comments

Comments

@RicardoM17
Copy link

Description

If we have a compose file with two services A and B, where B depends_on A then the following used to be the scenario.

  1. docker compose up -d would start both service-a and service-b as expected.

If afterwards we ran

  1. docker compose up -d service-b then nothing would happen. We'd just get container service-b Running.

This was true until at least Docker Compose version v2.15.1. However at least from Docker Compose version v2.26.0 when running command 2 above we recreate service-b even if nothing actually changed. This is also the case with the latest compose version v2.29.2.

  1. Running docker compose up -d will recreate service-b again which, again, didn't use to happen.

I had a look and I suspect that this happens because when starting a container with no-deps, the com.docker.compose.depends_on label is now empty, when before it wasn't. I think that this is the diff that causes the container to be recreated.

So this seems to be an unintended regression and it does make docker compose rather more finicky than before so hopefully it can be fixed.

Steps To Reproduce

No response

Compose Version

No response

Docker Environment

No response

Anything else?

No response

@ndeloof
Copy link
Contributor

ndeloof commented Aug 20, 2024

The root cause is that we use a mutated ServiceConfig with depends_on definition removed when running --no-deps, which changes the service config hash. I'm investigating for a fix

@RicardoM17
Copy link
Author

Thanks for the quick reply @ndeloof

If you do find a fix and compile a version that I can download from this repository I can help confirm that the issue is resolved from my side but it also seems rather simple to reproduce.

Let me know if I can help in other way. If not I'll just wait 🤞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants