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

env: fix handling of long lowerdir paths in newer kernel/mount versions #1164

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

davvid
Copy link
Collaborator

@davvid davvid commented Dec 28, 2024

The new "lowerdir+" option can be used to specify multiple layers when mounting. This avoids a limitation with the "lowerdir" option which is limited to a max of 256 characters.

Fixes: #968

The new "lowerdir+" option can be used to specify multiple layers
when mounting. This avoids a limitation with the "lowerdir" option which
is limited to a max of 256 characters.

Fixes: spkenv#968
Copy link
Collaborator

@rydrman rydrman left a comment

Choose a reason for hiding this comment

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

Thanks for this! I agree that it's an improvement regardless of the other related discussions on the issue.

@davvid
Copy link
Collaborator Author

davvid commented Dec 31, 2024

Thanks for reviewing. If you're comfortable merging now feel free. Otherwise, I'll wait until next week before merging in case anyone brings up any backwards-compat concerns.

I think we've all either left centos7-era distros behind or are using fuse so it doesn't seem like this should be an issue, tho.

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.

[BUG] OverlayFSWithRenders can fail abrupty on newer versions of linux
2 participants