Bug 1159595 - [y2-bootloader] Installed vms use a path-id for suspend instead of uuid.
[y2-bootloader] Installed vms use a path-id for suspend instead of uuid.
Status: CONFIRMED
Classification: openSUSE
Product: openSUSE Distribution
Classification: openSUSE
Component: YaST2
Leap 15.1
Other Other
: P5 - None : Normal (vote)
: ---
Assigned To: YaST Team
Jiri Srain
https://trello.com/c/8T149v5G
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2019-12-19 23:45 UTC by William Brown
Modified: 2020-01-17 09:12 UTC (History)
3 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---
jsrain: needinfo? (william.brown)


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description William Brown 2019-12-19 23:45:38 UTC
When the system is installed with default partitioning (no changes), which should use UUID's, suspend to disk in /etc/default/grub is configured with swap by a path-id instead.

This can lead to VM's that are unable to boot when their swap partition changes location (IE changing from virtio to scsi) or a hardware machine that changes pci paths in some way (relocation of a disk controller or disk controller replacement due to RMA etc).

This causes machines to be unbootable because the resume=/dev/disk/path-id can never be resolved and has an infinite timeout. The machine then must be forced off and the resolution steps are unclear

Work around is to add noresume at boot, and to edit /etc/default/grub post install to remove the resume option. 

See also: https://bugzilla.suse.com/show_bug.cgi?id=1159236
Comment 1 Jiri Srain 2019-12-20 07:24:19 UTC
While this should not be any hard to reproduce, any chance you could provide the installer logs?
Comment 2 Ancor Gonzalez Sosa 2020-01-02 10:58:56 UTC
Josef, why do we use path ids in bootloader?
Comment 3 Josef Reidinger 2020-01-02 11:35:37 UTC
(In reply to Ancor Gonzalez Sosa from comment #2)
> Josef, why do we use path ids in bootloader?

Issue is that uuid is not known in time when bootloader proposal is created. So in needs to be done later during write, but of course it overwrite user modifications.

See my workaround for one L3 - https://github.com/yast/yast-bootloader/pull/564/files

if we decide we want to do it, then I can merge it and forward to newer code streams.