Bug 592967
Summary: | Boot fails with LVM logging to file and / not on LV | ||
---|---|---|---|
Product: | Red Hat Enterprise Linux 5 | Reporter: | David Kovalsky <dkovalsk> |
Component: | lvm2 | Assignee: | Dave Wysochanski <dwysocha> |
Status: | CLOSED ERRATA | QA Contact: | Corey Marthaler <cmarthal> |
Severity: | high | Docs Contact: | |
Priority: | high | ||
Version: | 5.5 | CC: | agk, benl, coughlan, dwysocha, heinzm, jbrassow, joe.thornber, jturner, mbroz, prockai |
Target Milestone: | rc | Keywords: | Regression, ZStream |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | lvm2-2.02.56-12.el5 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2011-01-13 22:41:45 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: | 601079 |
Description
David Kovalsky
2010-05-17 13:56:57 UTC
Review create_toolcontext(). Are the liblvm requirements *really* different from the normal command line tool ones? What is the 'if (stored_errno)' test actually meant for, given that the function already returns NULL on failure? Should the field be cleared after operations we don't care about failing? I don't understand why we added this code in init_lvm(): if (stored_errno()) { destroy_toolcontext(cmd); return_NULL; } liblvm returns cmd in this case - it does not tear down the context. So the tools seem to have become more restrictive than liblvm, which is the bug IMO. We should revert the above code. I take comment #3 about reverting the code back. I agree with comment #2 and the IRC discussion between agk and kabi - we should call reset_lvm_errno(1) at various points in that create_toolcontext() for init functions that do not return an error or the error message is ignored. Perhaps the reset should go inside the specific init function. Two patches checked in upstream, one resolves this issue, and a second fixes a related init issue (if init_rand fails). Fixed in lvm2-2.02.56-12.el5. Testing mentioned in comment #10 passed in the latest rpm (lvm2-2.02.74-1.el5). Marking verified. [root@grant-01 tmp]# pvscan /tmp/log/bar/foo/coreys_fake_file.log: fopen failed: No such file or directory Logging initialised at Mon Nov 8 17:27:05 2010 Set umask to 0077 read_urandom: /dev/urandom: open failed: No such file or directory Wiping cache of LVM-capable devices Wiping internal VG cache Walking through all physical volumes PV /dev/sdc1 VG centipede lvm2 [54.49 GB / 54.49 GB free] PV /dev/sdc2 VG centipede lvm2 [54.49 GB / 54.49 GB free] PV /dev/sdc3 VG centipede lvm2 [54.48 GB / 54.48 GB free] PV /dev/sdc5 VG centipede lvm2 [54.49 GB / 54.49 GB free] PV /dev/sdc6 VG centipede lvm2 [54.48 GB / 54.48 GB free] PV /dev/sdb1 VG centipede lvm2 [40.87 GB / 40.87 GB free] PV /dev/sdb2 VG centipede lvm2 [40.87 GB / 40.87 GB free] PV /dev/sdb3 VG centipede lvm2 [40.87 GB / 40.87 GB free] PV /dev/sdb5 VG centipede lvm2 [40.88 GB / 40.88 GB free] PV /dev/sda2 VG VolGroup00 lvm2 [74.38 GB / 0 free] Total: 10 [510.30 GB] / in use: 10 [510.30 GB] / in no VG: 0 [0 ] Wiping internal VG cache 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-2011-0052.html |