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

Add support for volume Subpath option #11597

Closed
wants to merge 3 commits into from

Conversation

vvoland
Copy link

@vvoland vvoland commented Mar 11, 2024

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

milas and others added 3 commits March 8, 2024 17:51
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>
Copy link

codecov bot commented Mar 11, 2024

Codecov Report

Attention: Patch coverage is 41.66667% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 58.15%. Comparing base (4efb897) to head (dac1ff6).

Files Patch % Lines
pkg/api/dryrunclient.go 0.00% 6 Missing ⚠️
pkg/compose/push.go 0.00% 1 Missing ⚠️
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.
📢 Have feedback on the report? Share it here.

@@ -144,8 +141,9 @@ jobs:
- plugin
- standalone
engine:
- 26.0.0-rc2
Copy link
Contributor

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

@ndeloof
Copy link
Contributor

ndeloof commented Mar 18, 2024

please rebase so we can merge

@vvoland
Copy link
Author

vvoland commented Mar 18, 2024

We still need #11583 to be merged first

Copy link
Contributor

@jhrotko jhrotko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ndeloof
Copy link
Contributor

ndeloof commented Mar 22, 2024

I cherry-picked required changes into #11652

@ndeloof ndeloof closed this Mar 22, 2024
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.

None yet

4 participants