-
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
Add support for volume Subpath option #11597
Conversation
Additionally update some workflows in CI to add Moby v26 RC2 to the matrix and upgrade the various Docker actions. Signed-off-by: Milas Bowman <milas.bowman@docker.com>
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #11597 +/- ##
==========================================
- Coverage 58.20% 58.15% -0.05%
==========================================
Files 135 135
Lines 11611 11612 +1
==========================================
- Hits 6758 6753 -5
- Misses 4182 4186 +4
- Partials 671 673 +2 ☔ View full report in Codecov by Sentry. |
@@ -144,8 +141,9 @@ jobs: | |||
- plugin | |||
- standalone | |||
engine: | |||
- 26.0.0-rc2 |
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.
👍 we indeed should have tests ran with RC release before those get public
please rebase so we can merge |
We still need #11583 to be merged first |
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
I cherry-picked required changes into #11652 |
What I did
Allows to use moby/moby#45687 in compose
Related issue
(not mandatory) A picture of a cute animal, if possible in relation to what you did