-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Add image mount options #5755
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 image mount options #5755
Conversation
5e1ea8d to
1baccee
Compare
Codecov ReportAttention: Patch coverage is
❌ Your patch status has failed because the patch coverage (14.58%) is below the target coverage (50.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## master #5755 +/- ##
==========================================
- Coverage 59.25% 59.18% -0.08%
==========================================
Files 352 352
Lines 29496 29544 +48
==========================================
+ Hits 17479 17486 +7
- Misses 11042 11078 +36
- Partials 975 980 +5 |
Signed-off-by: Laurent Goderre <[email protected]>
1baccee to
04b1b4e
Compare
vvoland
left a comment
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
laurazard
left a comment
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
|
As a heads up, mounting with an image subpath doesn't survive a docker daemon restart currently (bug?): moby/moby#50999 -- crosslinking for visibility if others run into it. |
- What I did
Add support for image mount options. Supports moby/moby#48798
- How I did it
- How to verify it
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)