[Bug 959641] New: When installing on UEFI, yast proposes /boot partition unnecessarily for LVM+Encryption proposal.

older
[Bug 1032593] NFS Client Hangs on...

bugzilla_noreply＠novell.com

17 Dec 2015 17 Dec '15

21:55

http://bugzilla.suse.com/show_bug.cgi?id=959641 Bug ID: 959641 Summary: When installing on UEFI, yast proposes /boot partition unnecessarily for LVM+Encryption proposal. Classification: openSUSE Product: openSUSE Tumbleweed Version: 2015* Hardware: Other OS: Other Status: NEW Severity: Major Priority: P5 - None Component: Installation Assignee: yast2-maintainers@suse.de Reporter: rbrown@suse.com QA Contact: jsrain@suse.com CC: kukuk@suse.com Found By: --- Blocker: --- Created attachment 659793 --> http://bugzilla.suse.com/attachment.cgi?id=659793&action=edit y2logs Background information: http://lists.opensuse.org/opensuse-factory/2015-12/msg00071.html Most relevant post: http://lists.opensuse.org/opensuse-factory/2015-12/msg00257.html GOAL: To install Leap, Tumbleweed or SLE on a machine with LVM and Encryption and still have the 'Boot to Snapshot' feature operational. This should work fine on any disk with a GPT partition table. STEPS TO REPRODUCE: - Start with a blank disk with a GPT partition table (parted /dev/sda mklabel gpt) - Boot Tumbleweed, Leap 42.1, or SLE 12 SP1 installation media - On the Partition proposal screen, click 'Edit Partition Proposal' - Select "Create LVM-based Partition Proposal" and "Encrypt Volume Group" - Ensure that 'btrfs' is picked as the root filesystem and 'Enabled Snapshots' is selected - Click OK EXPECTED RESULTS: Proposal should include include an LVM Volume Group which contains / (root), /home and swap partitions There should not be a separate /boot partition outside the LVM Volume Group ACTUAL RESULTS: If the system boots to BIOS: Proposal *MATCHES* Expected Results. You end up with a proposal that includes a small Grub/BIOS partition, a large LVM partition, and an LVM Volume Group which contains /, /home, and swap There is no separate /boot partition outside of the LVM Volume Group If the system boots to UEFI (or UEFI Secure Boot): Proposal *DOES NOT MATCH* Expected Results Proposal consists of a small (156MB) /boot/efi partition, a slightly larger (400MB) /boot partition, and then the remainder of the disk is an LVM Volume Group which contains /, /home, and swap CONSEQUENCE: On UEFI systems, the LVM + Encryption partition proposals presented by YaST result in a system which has a separate /boot partition and therefore a broken 'Boot to Snapshot' Feature YaST should not do this, as there does not seem to be any technical reason to enforce a separate /boot partition when using LVM+Encryption. y2logs from a Leap 42.1 UEFI installation are attached -- You are receiving this mail because: You are on the CC list for the bug.

Attachments:

attachment.htm (text/html — 4.6 KB)

Show replies by date

bugzilla_noreply＠novell.com

18 Dec 18 Dec

09:35

New subject: [Bug 959641] When installing on UEFI, yast proposes /boot partition unnecessarily for LVM+Encryption proposal.

http://bugzilla.suse.com/show_bug.cgi?id=959641 http://bugzilla.suse.com/show_bug.cgi?id=959641#c1 Ancor Gonzalez Sosa changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ancor@suse.com, | |aschnell@suse.com Flags| |needinfo?(aschnell@suse.com | |) --- Comment #1 from Ancor Gonzalez Sosa --- Arvin, is this realistically fixable with the current code base? -- You are receiving this mail because: You are on the CC list for the bug.

bugzilla_noreply＠novell.com

09:46

New subject: [Bug 959641] When installing on UEFI, yast proposes /boot partition unnecessarily for LVM+Encryption proposal.

http://bugzilla.suse.com/show_bug.cgi?id=959641 http://bugzilla.suse.com/show_bug.cgi?id=959641#c2 Thorsten Kukuk changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fcrozat@suse.com --- Comment #2 from Thorsten Kukuk --- (In reply to Ancor Gonzalez Sosa from comment #1)

...

Arvin, is this realistically fixable with the current code base?

It works for GPT tables with legacy BIOS today. So the code is there and works. We only need to adjust the check for UEFI and GPT tables. -- You are receiving this mail because: You are on the CC list for the bug.

bugzilla_noreply＠novell.com

10:45

New subject: [Bug 959641] When installing on UEFI, yast proposes /boot partition unnecessarily for LVM+Encryption proposal.

http://bugzilla.suse.com/show_bug.cgi?id=959641 http://bugzilla.suse.com/show_bug.cgi?id=959641#c3 Arvin Schnell changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rbrown@suse.com Flags|needinfo?(aschnell@suse.com |needinfo?(rbrown@suse.com) |) | --- Comment #3 from Arvin Schnell --- https://bugzilla.suse.com/show_bug.cgi?id=887888#c37 shows that the extra /boot partition was needed. If the bootloader is now per default capable to handle the situation please make a feature request to change the proposal. Then it is documented and gets tested. -- You are receiving this mail because: You are on the CC list for the bug.

bugzilla_noreply＠novell.com

10:51

New subject: [Bug 959641] When installing on UEFI, yast proposes /boot partition unnecessarily for LVM+Encryption proposal.

http://bugzilla.suse.com/show_bug.cgi?id=959641 http://bugzilla.suse.com/show_bug.cgi?id=959641#c4 --- Comment #4 from Thorsten Kukuk --- (In reply to Arvin Schnell from comment #3)

...

https://bugzilla.suse.com/show_bug.cgi?id=887888#c37 shows that the extra /boot partition was needed.

No, the opposite: this comment is from me and documents, what for grub2 needs to be changed that it works without extra /boot partition. And snwint fixed it afterwards in perl-Bootloader. -- You are receiving this mail because: You are on the CC list for the bug.

bugzilla_noreply＠novell.com

10:56

New subject: [Bug 959641] When installing on UEFI, yast proposes /boot partition unnecessarily for LVM+Encryption proposal.

http://bugzilla.suse.com/show_bug.cgi?id=959641 http://bugzilla.suse.com/show_bug.cgi?id=959641#c5 --- Comment #5 from Arvin Schnell --- Before the change it *was* needed. And nobody included in the bug took care that the proposal was changed. -- You are receiving this mail because: You are on the CC list for the bug.

bugzilla_noreply＠novell.com

22:43

New subject: [Bug 959641] When installing on UEFI, yast proposes /boot partition unnecessarily for LVM+Encryption proposal.

http://bugzilla.suse.com/show_bug.cgi?id=959641 http://bugzilla.suse.com/show_bug.cgi?id=959641#c6 Richard Brown changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(rbrown@suse.com) | --- Comment #6 from Richard Brown --- (In reply to Arvin Schnell from comment #3)

...

https://bugzilla.suse.com/show_bug.cgi?id=887888#c37 shows that the extra /boot partition was needed. If the bootloader is now per default capable to handle the situation please make a feature request to change the proposal. Then it is documented and gets tested.

As confirmed by Thorsten in Comment #4, the details from the previous bug ( https://bugzilla.suse.com/show_bug.cgi?id=887888#c37 ) show that a /boot partition is not needed This is further confirmed by the fact that this works perfectly fine in the Legacy BIOS proposal already provided by YaST YaST supports LVM+Encryption with BTRFS without a /boot partition on Legacy BIOS, and it should do so on UEFI also. I therefore consider this a bug and not worthy of a feature request until a time where someone explains why /boot is needed in UEFI (but clearly doesn't in any other case) (In reply to Arvin Schnell from comment #5)

...

Before the change it *was* needed. And nobody included in the bug took care that the proposal was changed.

It looks to me like someone took care of the Legacy BIOS proposal but not the UEFI proposal, leading to this bug.. -- You are receiving this mail because: You are on the CC list for the bug.

bugzilla_noreply＠novell.com

5 Jan 5 Jan