-
Notifications
You must be signed in to change notification settings - Fork 49
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
Partition mode #189
Partition mode #189
Conversation
9a973fb
to
b568847
Compare
This is ready to review. It is currently vendored into osbuild/osbuild-composer#3723 to see if the full suite of tests will pass with it. It's slightly different from the original PR, I decided to make an empty/missing PartitioningMode use DefaultPartitioningMode so that the string conversions are nicer to read. The fact that Default results in AutoLVM is enforced in this code and nowhere else. eg. If the osbuild-composer code gets an empty entry it calls this code with DefaultPartitioningMode instead of AutoLVM so that we have a single place deciding what the default actually is. |
Awesome, @bcl! I will wait for Achilleas to return, I don't feel comfortable giving this an approval, considering that I wrote most of these changes. :) |
b568847
to
57a49df
Compare
57a49df
to
5f20973
Compare
pt.features() now returns a struct with flags set for each feature used in the members of the partition table. Also adds a test for the new function. Based on PR #3222
This adds support for AutoLVM mode which uses LVM when there are 1 or more extra mountpoints, LVM mode which always uses LVM, and Raw mode which always uses raw partitions. This also removes a panic from ensureLVM, returning an error instead. There are cases where the mode can conflict with the selected partition table. Also includes a new test for always using LVM.
And use it in the distro's getPartitionTable. TODO Logic about that combination of things is or is not supported needs a more central location than embedded in the functions.
5f20973
to
117134d
Compare
THANKS! <3 |
This is basically a port of osbuild/osbuild-composer#3222 to images. I think this is complete, but I need to clean up the osbuild-composer part first so marking this as a draft for now.