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

btrfs-progs missing during mkinitcpio generation #1411

Closed
Torxed opened this issue Aug 9, 2022 · 4 comments · Fixed by #1412
Closed

btrfs-progs missing during mkinitcpio generation #1411

Torxed opened this issue Aug 9, 2022 · 4 comments · Fixed by #1412
Labels
bug Something isn't working

Comments

@Torxed
Copy link
Member

Torxed commented Aug 9, 2022

Strictly speaking this is an error that results in exit code 256 (1), but the image will still be generated.
But it causes archinstall to react on the exit code and aborts the installation.

Either we have to be more tolerant towards mkinitcpio and allow "bad" exit codes and check for Image generation successful, or we have to include btrfs-progs earlier in the steps.

screenshot

@Torxed Torxed added the bug Something isn't working label Aug 9, 2022
@Torxed
Copy link
Member Author

Torxed commented Aug 9, 2022

This logic is broken:

for partition in self.partitions:
if partition.filesystem == 'btrfs':
# if partition.encrypted:
if 'btrfs-progs' not in self.base_packages:
self.base_packages.append('btrfs-progs')

Because at the time of:

self.pacstrap(self.base_packages)

The following packages are installed: base base-devel linux-firmware linux
screenshot

Which means this will "fail":

self.mkinitcpio('-P')

Verifiable with:

import archinstall
archinstall.get_partitions_in_use('/mnt/archinstall')
{}

This was referenced Aug 9, 2022
@Torxed Torxed linked a pull request Aug 9, 2022 that will close this issue
1 task
@Torxed
Copy link
Member Author

Torxed commented Aug 9, 2022

Trying to think of the best approach forward here.
Because technically those paths are on that partition, so that's not a lie.
But that partition doesn't have a mountpoint, because the subvolumes do.

@Torxed
Copy link
Member Author

Torxed commented Aug 9, 2022

Easy tweak, I check if installation.target is inside the Partition.mountpoints instead of .mountpoint as we do recieve all the mountpoints from that function, including subvolumes.

@Torxed
Copy link
Member Author

Torxed commented Aug 9, 2022

All in all, it was an error in detecting partitions in use. Because of pathlib.Path being introduced, and MapperDev missing a .mountpoints function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant