Skip to content

cri: use mount manager when image has volumes#12847

Merged
dmcgowan merged 1 commit into
containerd:mainfrom
Champ-Goblem:fix/cri-withvolume-use-mount-manager
Feb 7, 2026
Merged

cri: use mount manager when image has volumes#12847
dmcgowan merged 1 commit into
containerd:mainfrom
Champ-Goblem:fix/cri-withvolume-use-mount-manager

Conversation

@Champ-Goblem

Copy link
Copy Markdown
Contributor

When a container image has a volumes attribute, currently the mount manager is not used here. This leads to issues with the EROFS snapshotter, notably that rwlayer.img is not present when the snapshotter is configured to create a writable block volume.

This change implements mount manager as part of the image volumes processing, so that mount manager mounts are correctly processes before calling mount.All

Fixes: #12834

When a container image has a volumes attribute, currently the mount manager is not used here. This leads to issues with the EROFS snapshotter, notably that rwlayer.img is not present when the snapshotter is configured to create a writable block volume.

This change implements mount manager as part of the image volumes processing, so that mount manager mounts are correctly processes before calling mount.All

Fixes: containerd#12834
Signed-off-by: Champ-Goblem <[email protected]>
@github-project-automation github-project-automation Bot moved this to Needs Triage in Pull Request Review Feb 2, 2026
@dosubot dosubot Bot added the area/cri Container Runtime Interface (CRI) label Feb 2, 2026

@hsiangkao hsiangkao 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

@hsiangkao
hsiangkao requested review from dmcgowan and fuweid February 3, 2026 01:49

@dmcgowan dmcgowan 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.

This is another area where we should reconsider the interface to use fs.FS rather than assuming the host can/should mount to perform this operation.

@github-project-automation github-project-automation Bot moved this from Needs Triage to Review In Progress in Pull Request Review Feb 3, 2026
@dmcgowan
dmcgowan added this pull request to the merge queue Feb 3, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 3, 2026
@hsiangkao

Copy link
Copy Markdown
Member

@containerd/committers (... could someone try again?..)

@dmcgowan
dmcgowan added this pull request to the merge queue Feb 7, 2026
Merged via the queue into containerd:main with commit b8a1501 Feb 7, 2026
54 checks passed
@github-project-automation github-project-automation Bot moved this from Review In Progress to Done in Pull Request Review Feb 7, 2026
@hsiangkao

Copy link
Copy Markdown
Member

/cherry-pick release/2.2

@k8s-infra-cherrypick-robot

Copy link
Copy Markdown

@hsiangkao: new pull request created: #12874

Details

In response to this:

/cherry-pick release/2.2

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@hsiangkao hsiangkao added the cherry-pick/2.2.x Change to be cherry picked to release/2.2 branch label Feb 9, 2026
@hsiangkao

Copy link
Copy Markdown
Member

/cherry-pick release/2.2

@k8s-infra-cherrypick-robot

Copy link
Copy Markdown

@hsiangkao: new pull request created: #13242

Details

In response to this:

/cherry-pick release/2.2

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@fuweid fuweid added cherry-picked/2.2.x PR commits are cherry-picked into release/2.2 branch and removed cherry-pick/2.2.x Change to be cherry picked to release/2.2 branch labels Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cri Container Runtime Interface (CRI) cherry-picked/2.2.x PR commits are cherry-picked into release/2.2 branch size/S

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

EROFS: rwlayer.img failure when the docker image has a Volumes attribute

7 participants