1) Containers don't have formats, so this checks always fails, making
mdraid BIOS RAID sets not show up in storage.partitioned.
2) MDRaidArrayDevice.biosraid is used to not show biosraid related
mdraid sets as editable raid sets in the partition UI, so we should return
true for containers too, as we don't want to show those there either
3) Containers however are not partitionable, so we cannot use the
biosraid property in the partitionable property.
---
storage/devices.py | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/storage/devices.py b/storage/devices.py
index f21e730..3bae2d8 100644
--- a/storage/devices.py
+++ b/storage/devices.py
@@ -2824,13 +2824,13 @@ class MDRaidArrayDevice(StorageDevice):
@property
def biosraid(self):
- return (len(self.devices) != 0 and
- self.devices[0].type == "mdcontainer" and
- getattr(self.devices[0].format, "biosraid", False))
+ """ Is this a BIOS RAID related set? """
+ return self.type == "mdcontainer" or \
+ (self.devices and self.devices[0].type == "mdcontainer")
@property
def partitionable(self):
- return self.biosraid
+ return self.devices and self.devices[0].type == "mdcontainer"
def dracutSetupString(self):
return "rd_MD_UUID=%s" % self.uuid
--
1.6.5.2
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
