Bug 250631
Summary: | [RHEL5.1 Beta] booty: not using LABEL for non-dm devices causes boot failure | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 5 | Reporter: | Kiyoshi Ueda <kueda> | ||||||
Component: | booty | Assignee: | David Cantrell <dcantrell> | ||||||
Status: | CLOSED ERRATA | QA Contact: | |||||||
Severity: | high | Docs Contact: | |||||||
Priority: | low | ||||||||
Version: | 5.1 | CC: | i-kitayama, jlaska, jnomura, junichi.nomura, kueda, kueda, tao | ||||||
Target Milestone: | --- | Keywords: | OtherQA, Regression | ||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | RHBA-2007-0748 | Doc Type: | Bug Fix | ||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2007-11-07 17:59:27 UTC | Type: | --- | ||||||
Regression: | --- | Mount Type: | --- | ||||||
Documentation: | --- | CRM: | |||||||
Verified Versions: | Category: | --- | |||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||
Embargoed: | |||||||||
Bug Depends On: | |||||||||
Bug Blocks: | 228988 | ||||||||
Attachments: |
|
Description
Kiyoshi Ueda
2007-08-02 16:00:50 UTC
Just a forwarder: Event posted 08-07-2007 07:03am by kueda-RH I tested booty-0.80.4-4.IT128375.1.noarch.rpm, but it used label when multipath device is used for root volume. So the fix is not correct, it may cause other boot failure in case of multipath root. The patch included in it checks existence of '/mpath' in the label string of the device to see if the device is multipath or not. But, the label of multipath devices is not special like including /mpath. It is similar to non-multipath devices. So it has to check the device name, not the label string. Also, since the patch is made based on older booty (0.80.4-1), it doesn't apply cleanly to the latest booty-0.80.4-4. I modified the patch (attached) to fix the issues above. I confirmed it worked for all of the following cases: [LABEL should be used: "root=LABEL=..."] 1. SCSI + noLVM 2. FC + nompath + noLVM [LABEL should NOT be used: "root=/dev/..."] 3. SCSI + LVM 4. FC + nompath + LVM 5. FC + mpath + noLVM 6. FC + mpath + LVM diff -rup booty-0.80.4-4/bootloaderInfo.py booty-0.80.4-4.fix/bootloaderInfo.py --- booty-0.80.4-4/bootloaderInfo.py 2007-08-06 12:42:48.000000000 -0400 +++ booty-0.80.4-4.fix/bootloaderInfo.py 2007-08-06 12:44:05.000000000 -0400 @@ -2061,7 +2061,7 @@ def getRootDevName(initrd, fsset, rootDe try: rootEntry = fsset.getEntryByMountPoint("/") - if rootEntry.getLabel() is not None and rootEntry.find('/mpath') == -1: + if rootEntry.getLabel() is not None and rootEntry.device.getDevice().find('/mpath') == -1: return "LABEL=%s" %(rootEntry.getLabel(),) return "/dev/%s" %(rootDev,) except: Internal Status set to 'Waiting on SEG' This event sent from IssueTracker by tumeya issue 128375 A comment from SEG: New test packages with the fix suggested. Keep in mind I'm not adding a second patch to change the same line, but actually updating the patch that changes that line in order to fix it. So although you see the same patches in these packages, the contents of booty-0.80.4-multipath.patch will be updated. Version changed from '5.1' to 'UNKNOWN' This event sent from IssueTracker by tumeya issue 128375 Created attachment 160794 [details]
booty from SEG
Created attachment 160795 [details]
booty from SEG
The fix of the updated booty-0.80.4-multipath.patch in Comment#6 is same as the fix in Comment#2. So the test results in Comment#2 is still valid. This problem is a regression from RHEL5.0. Please apply the fix. > Version changed from '5.1' to 'UNKNOWN'
Will the fix be included in 5.1?
Please note that this is a regression from 5.0.
Adding devel-ack and reassigning to me since I broke this in the first place. Fix from comment #2 verified to work. Updating booty and rebuilding for 5.1. The fix will be in booty-0.80.4-5. Setting bug to MODIFIED. An advisory has been issued which should help the problem described in this bug report. This report is therefore being closed with a resolution of ERRATA. For more information on the solution and/or where to find the updated files, please follow the link below. You may reopen this bug report if the solution does not work for you. http://rhn.redhat.com/errata/RHBA-2007-0748.html |