Skip to content

[release/1.7] Support both styles of volatile mount option#13299

Merged
mxpv merged 1 commit into
containerd:release/1.7from
chrishenzie:release/1.7-volatile
Apr 28, 2026
Merged

[release/1.7] Support both styles of volatile mount option#13299
mxpv merged 1 commit into
containerd:release/1.7from
chrishenzie:release/1.7-volatile

Conversation

@chrishenzie

@chrishenzie chrishenzie commented Apr 27, 2026

Copy link
Copy Markdown
Member

Manual backport of #13256.

Support both "volatile" and "fsync=volatile" mount options for volatile snapshotter

Kernel 6.12.80+ returns 'fsync=volatile' instead of just 'volatile'
in mount options, which breaks containerd's exact string matching
checks.

Fixes this issue by adding support for 'fsync=volatile' in addition
to the existing 'volatile' check in RemoveVolatileOption and
addVolatileOptionOnImageVolumeMount.

Assisted-by: Antigravity
Signed-off-by: Chris Henzie <chrishenzie@gmail.com>
(cherry picked from commit 93f7a62)
Signed-off-by: Chris Henzie <chrishenzie@gmail.com>
@github-project-automation github-project-automation Bot moved this to Needs Triage in Pull Request Review Apr 27, 2026
@samuelkarp samuelkarp changed the title Support both styles of volatile mount option [release/1.7] Support both styles of volatile mount option Apr 27, 2026
@chrishenzie chrishenzie marked this pull request as ready for review April 28, 2026 15:41
@chrishenzie chrishenzie requested a review from samuelkarp as a code owner April 28, 2026 15:41
@dosubot dosubot Bot added the area/runtime Runtime label Apr 28, 2026
@github-project-automation github-project-automation Bot moved this from Needs Triage to Review In Progress in Pull Request Review Apr 28, 2026
@mxpv mxpv merged commit 4393e22 into containerd:release/1.7 Apr 28, 2026
89 of 92 checks passed
@github-project-automation github-project-automation Bot moved this from Review In Progress to Done in Pull Request Review Apr 28, 2026
@chrishenzie chrishenzie deleted the release/1.7-volatile branch April 28, 2026 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants