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.

Sign up for GitHub

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

Jump to bottom

Add support for configurable LVM metadata size #1066

Closed
stgraber opened this issue Aug 1, 2024 · 0 comments
Closed

Add support for configurable LVM metadata size #1066

stgraber opened this issue Aug 1, 2024 · 0 comments
Assignees
@stgraber
Labels
API Changes to the REST API Documentation Documentation needs updating Easy Good for new contributors Feature New feature, not a bug
Milestone
incus-6.5

Comments

@stgraber
Copy link
Member

stgraber commented Aug 1, 2024

When dealing with large volume groups, the default metadata size of 1MiB quickly becomes a problem.
As LVM doesn't scale that value automatically based on PV size and it's not something that can be resized after the fact, it'd be good to have a (global) lvm.metadata_size as an analog to lvm.thinpool_metadata_size but for the VG metadata size itself.
@stgraber stgraber added Documentation Documentation needs updating Feature New feature, not a bug Easy Good for new contributors API Changes to the REST API labels Aug 1, 2024
@stgraber stgraber modified the milestones: soon, incus-6.5 Aug 1, 2024
@stgraber stgraber self-assigned this Aug 29, 2024
stgraber added a commit to stgraber/incus that referenced this issue Sep 1, 2024
@stgraber
incusd/storage/lvm: Add lvm.metadata_size
c600065 
Closes lxc#1066

Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
@hallyn hallyn closed this as completed in 2bc2f69 Sep 2, 2024
dkwo pushed a commit to dkwo/incus that referenced this issue Sep 2, 2024
@stgraber @dkwo
incusd/storage/lvm: Add lvm.metadata_size
09d0e1e 
Closes lxc#1066

Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
stgraber added a commit that referenced this issue Sep 2, 2024
@stgraber
incusd/storage/lvm: Add lvm.metadata_size
10711ea 
Closes #1066

Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stgraber stgraber

Labels
API Changes to the REST API Documentation Documentation needs updating Easy Good for new contributors Feature New feature, not a bug
Milestone
incus-6.5
Development

No branches or pull requests
1 participant
@stgraber