-
Notifications
You must be signed in to change notification settings - Fork 63
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
Introduce new disk partitioning scheme #478
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see the detailed comments.
@unbel13ver can you provide test instructions for this PR? What is the expected outcome of new partitioning scheme? |
68c2e84
to
00b7412
Compare
Testing instructions added. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- lsblk gives expected output
- ci-test-automation passed
- performance ok
- apps launch
00b7412
to
af1d8c7
Compare
af1d8c7
to
1794271
Compare
With the latest commit 9d7ab74: |
Reason for limited size is probably because of disko-basic-postboot commands didn't work as expected due to change in partitioning scheme. |
Yes, thanks for noticing this. The |
I am not sure the issue is related to this PR. NixOS picks the device to boot by its label, and in this case there are two devices with the same label ( |
New disk configuration provides grounds for upcoming features, such as AB software updates and Storage VM and many more. Signed-off-by: Ivan Nikolaenko <[email protected]>
ZFS partition is now extended during the first boot.
I have also renamed zfs pool from |
Tested on Lenovo-X1Everything seems to be good with both installer and debug-image! (I did not test nixos-rebuild though, please let me know if that needs to be still checked)
(As a note for our testing team: The free space in host is reduced with this PR and there is no longer an excessive amount of free space to run fileio-tests. We need to decide how we want to proceed with that.) |
New disk configuration provides grounds for upcoming features, such as AB software updates and Storage VM and many more.
The Lenovo X1 config has two LVM pools, first one is fixed-size 250G "system" partition (which is going to be encrypted) and the rest of the disk is dedicated to the StorageVM.
Description of changes
Checklist for things done
x86_64
aarch64
riscv64
nix flake check --accept-flake-config
and it passesTesting
Current upstream partition scheme is as follows:
To test new partitioning scheme, run
lenovo-x1-carbon-gen11-debug-installer
image and install Ghaf into the internal SSD storage of Lenovo-X1.After insallation is completed and the laptop is booted into the Ghaf system, check partitions with
lsblk
command.It should be as follows:
Check zpool is online:
Check datasets are mounted successfully:
Optionally - check there are no failed
systemd
units: