Skip to content

Add image mount options - #5755

Merged
vvoland merged 1 commit into
docker:masterfrom
LaurentGoderre:image-mount-subpath
Feb 13, 2025
Merged

Add image mount options#5755
vvoland merged 1 commit into
docker:masterfrom
LaurentGoderre:image-mount-subpath

Conversation

@LaurentGoderre

@LaurentGoderre LaurentGoderre commented Jan 15, 2025

Copy link
Copy Markdown
Member

- 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

Add `--mount type=image,image-subpath=[subpath],...` option to mount a specific path from the image.

- A picture of a cute animal (not mandatory but encouraged)

@LaurentGoderre
LaurentGoderre force-pushed the image-mount-subpath branch 3 times, most recently from 5e1ea8d to 1baccee Compare February 5, 2025 19:48
@codecov-commenter

codecov-commenter commented Feb 5, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 14.58333% with 41 lines in your changes missing coverage. Please review.

Project coverage is 59.18%. Comparing base (1e3e1bd) to head (04b1b4e).
Report is 8 commits behind head on master.

❌ 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     
@LaurentGoderre
LaurentGoderre marked this pull request as ready for review February 5, 2025 19:53
@vvoland vvoland added this to the 28.0.0 milestone Feb 6, 2025
Signed-off-by: Laurent Goderre <laurent.goderre@docker.com>

@vvoland vvoland left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@laurazard laurazard left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@vvoland
vvoland merged commit c9e7dae into docker:master Feb 13, 2025
@LaurentGoderre
LaurentGoderre deleted the image-mount-subpath branch February 13, 2025 14:25
@thaJeztah thaJeztah mentioned this pull request May 1, 2025
@apollo13

Copy link
Copy Markdown

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment