Bug 467289
| Summary: | we are storing error strings as fs labels because execWithCapture is broken | ||
|---|---|---|---|
| Product: | Red Hat Enterprise Linux 5 | Reporter: | David Lehman <dlehman> |
| Component: | anaconda | Assignee: | David Lehman <dlehman> |
| Status: | CLOSED ERRATA | QA Contact: | Alexander Todorov <atodorov> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | 5.3 | CC: | atodorov, borgan, ddumas, syeghiay, tao |
| Target Milestone: | rc | ||
| Target Release: | --- | ||
| Hardware: | ia64 | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2009-01-20 21:36:43 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: | |||
For additional info & partitioning layout see: https://bugzilla.redhat.com/show_bug.cgi?id=456283#c28 Marking as release blocker because this prevents functioning of rescue mode on IA64 This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux maintenance release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux Update release for currently deployed products. This request is not yet committed for inclusion in an Update release. Fix will be in anaconda-11.1.2.149-1. I can't reproduce this issue neither with partitioning layout described in bug #456283 comment #28 neither with the layout described in comment #0 With an older release on encrypted raid5 array shows the error: # python >>> import isys >>> isys.readFSLabel("/dev/md0", 0) 'Logical sector size is zero.' Trying with snap #2 leads to: >>> import isys >>> label = isys.readFSLabel("/dev/md0", 0) >>> label is None True Moving to VERIFIED. Just for the record: rescue mode not always fails. *** Bug 427457 has been marked as a duplicate of this bug. *** 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 therefore 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-2009-0164.html |
Description of problem: We're storing error strings as filesystem labels. This specific manifestation is a call to iutil.execWithCapture from isys._readFATLabel(). On ia64 systems with multiple PVs *or* multiple VGs, the following error dialog will halt any attempt to bring up the system in rescue mode: +----------------------+ Duplicate Labels +-----------------------+ | | | Multiple devices on your system are labelled Logical sector | | size is zero.. Labels across devices must be unique for your | | system to function properly. | | | | Please fix this problem and restart the installation process. | | | | +--------+ | | | Reboot | | | +--------+ | | | | | +-----------------------------------------------------------------+ The basic problem is that iutil.execWithCapture makes no use of the stderr input argument and instead always dupes stderr to stdout. So, when dosfslabel throws an error to stdout trying to read a fat label from a PV, we end up thinking the PV has a label of "Logical sector size is zero." Another one is a VG with a label of "Read 512 bytes at 0:Is a directory". Callers who want stderr duped to stdout should use execWithCapture(...,stderr=1). Version-Release number of selected component (if applicable): anaconda-11.1.2.136-2 How reproducible: Always Steps to Reproduce: 1. create a partition that doesn't contain any fslabel 2. call isys._readFATLabel (or, on ia64, isys.readFSLabel) on the device -OR- 1. try to rescue an ia64 box containing more than one PV or more than one VG Actual results: An error string is returned as though it was a label, or an error indicating rescue mode is aborting. Expected results: No return value if no label is present, and successful rescue of systems we installed. Additional info: This only causes problems on ia64, but that's only because we don't collect FAT labels on other arches.