Red Hat Bugzilla – Bug 463780
Add new logical volume raises exception
Last modified: 2009-01-20 16:37:35 EST
Created attachment 317608 [details]
Description of problem:
In partition editor after "Remove all partitions on selected
drives and create default layout", trying to add a logical
partition to VolGroup00 raises exception.
Version-Release number of selected component (if applicable):
tree: rhel5 20080919 nightly Server i386
Steps to Reproduce:
1. in partitioning select
"Remove all partitions on selected drives and create default layout"
and check "Review and modify partitioning layout", go "Next"
2. in partittion editor edit LogVol00 partition
3. in "Edit LVM Volume Group" dialog delete LogVol00 partition
4. in "Edit LVM Volume Group" dialog click "Add"
Traceback (most recent call first):
File "/usr/lib/anaconda/partRequests.py", line 981, in getActualSize
retval = lvm.clampLVSizeRequest(retval, vgreq.pesize)
File "/usr/lib/anaconda/iw/lvm_dialog_gui.py", line 445, in editLogicalVolume
sizeEntry.set_text("%Ld" % (logrequest.getActualSize(self.partitions, self.diskset),))
File "/usr/lib/anaconda/iw/lvm_dialog_gui.py", line 775, in addLogicalVolumeCB
self.editLogicalVolume(request, isNew = 1)
AttributeError: 'NoneType' object has no attribute 'pesize'
Dialog for adding should appear
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
Created attachment 317675 [details]
patch with fix
Seems like to be caused by commit 4aa9ca1c35b867fa5a4d94c41591700ca7ab5edb,
The patch should solve this bug while keeping what was intended with patch
for #415871 in.
(In reply to comment #2)
> ............................ while keeping what was intended with patch
> for #415871 in.
Joel, could you review attached patch that wrt ^^^^?
Created attachment 317677 [details]
updated patch with fix
To consider the "round down patch, 4aa9ca1c35b867fa5a4d94c41591700ca7ab5edb" only for the volumes that are described with percent seems OK to me.
I'd go ahead and commit it.
*** Bug 463843 has been marked as a duplicate of this bug. ***
The patch from comment #4 doesn't work.
Things seem pretty more complicated.
Created attachment 318049 [details]
should fix the bug, also tested against reproducer of original bug #415871
Fixed with commit 08233b0c42f8b453ff7d8bde03c9adc57d92d7ed,
should go into anaconda-126.96.36.199.
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.