[PATCH rhel5] Fix handling of existing PVs on disabled disk (#532536).

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Traceback occurs when installing on system with existing VG having PVs
on 2 drives if one drive is unselected for installation.

Traceback (most recent call first):
  File "/usr/lib/anaconda/partRequests.py", line 811, in getDevice
    if (r.size > 0) or (r.device is not None):
  File "/usr/lib/anaconda/partRequests.py", line 202, in toEntry
    device = self.getDevice(partitions)
  File "/usr/lib/anaconda/partitioning.py", line 124, in partitioningComplete
    entry = request.toEntry(anaconda.id.partitions)
  File "/usr/lib/anaconda/dispatch.py", line 201, in moveStep
    rc = stepFunc(self.anaconda)
  File "/usr/lib/anaconda/dispatch.py", line 124, in gotoNext
    self.moveStep()
  File "/usr/lib/anaconda/gui.py", line 1156, in nextClicked
    self.anaconda.dispatch.gotoNext()
AttributeError: 'NoneType' object has no attribute 'size'
---
 partRequests.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/partRequests.py b/partRequests.py
index 0ae6b74..809fb5e 100644
--- a/partRequests.py
+++ b/partRequests.py
@@ -808,7 +808,7 @@ class VolumeGroupRequestSpec(RequestSpec):
             r = partitions.getRequestByID(pv)
             # a size of zero implies we did autopartitioning of
             # pvs everywhere we could
-            if (r.size > 0) or (r.device is not None):
+            if r and ((r.size > 0) or (r.device is not None)):
                 pvs.append(r.getDevice(partitions))
         self.dev = fsset.VolumeGroupDevice(self.volumeGroupName, pvs,
                                            self.pesize,
-- 
1.6.0.6

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux