-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Implement depends_on #2708
Implement depends_on #2708
Conversation
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Signed-off-by: Daniel Nephin <dnephin@docker.com>
Signed-off-by: Daniel Nephin <dnephin@docker.com>
LGTM |
Just stumbled upon this merge :) TL;dr: |
That's the idea, yes |
Are there any instruction anywhere on how to install the latest compose so I can test this? |
@cescoferraro RC2 will be out with this soon. |
Cherry-picked into #2753 |
I have struggled with this for so long 😭😭 @aanand Great job! |
Fixes #2682
Second and third commit contain some cleanup for things I noticed while working on it.
Services will be ordered by the
depends_on
key.