-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
don't rely on depends_on to resolve volume_from, better use observed state #10999
Conversation
6eefef3
to
81b8ba6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #10999 +/- ##
==========================================
+ Coverage 57.88% 57.95% +0.07%
==========================================
Files 129 129
Lines 11160 11108 -52
==========================================
- Hits 6460 6438 -22
+ Misses 4063 4038 -25
+ Partials 637 632 -5
☔ View full report in Codecov by Sentry. |
ad3dc84
to
bf961a9
Compare
…state Signed-off-by: Nicolas De Loof <[email protected]>
273c702
to
e2aa036
Compare
Signed-off-by: Nicolas De Loof <[email protected]>
e2aa036
to
92637d2
Compare
What I did
remove preemptive parsing of
volumes_from
doing convergence based on known dependencies, better rely on observed state to select target container. This also removes the assumption we will be able to use container number1
Related issue
closes #10996
closes #10997
(not mandatory) A picture of a cute animal, if possible in relation to what you did