We don't really want the tree populated when we instantiate it, so don't populate it until someone calls its populate method. --- storage/__init__.py | 2 ++ storage/devicetree.py | 4 +--- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/storage/__init__.py b/storage/__init__.py index 51a892d..4d8c13b 100644 --- a/storage/__init__.py +++ b/storage/__init__.py @@ -167,6 +167,7 @@ class Storage(object): self.defaultFSType = get_default_filesystem_type() self.defaultBootFSType = get_default_filesystem_type(boot=True) + # these will both be empty until our reset method gets called self.devicetree = DeviceTree(intf=self.anaconda.intf, ignored=self.ignoredDisks, exclusive=self.exclusiveDisks, @@ -216,6 +217,7 @@ class Storage(object): zeroMbr=self.zeroMbr, passphrase=self.encryptionPassphrase, luksDict=self.__luksDevs) + self.devicetree.populate() self.fsset = FSSet(self.devicetree) w.pop() diff --git a/storage/devicetree.py b/storage/devicetree.py index 1692b0e..7bd9ecc 100644 --- a/storage/devicetree.py +++ b/storage/devicetree.py @@ -170,8 +170,6 @@ class DeviceTree(object): if luksDict and isinstance(luksDict, dict): self.__luksDevs = luksDict - self._populate() - def pruneActions(self): """ Prune loops and redundant actions from the queue. """ # handle device destroy actions @@ -1101,7 +1099,7 @@ class DeviceTree(object): log.info("setup of %s failed: %s" % (lv_device.name, e)) - def _populate(self): + def populate(self): """ Locate all storage devices. """ # each iteration scans any devices that have appeared since the # previous iteration -- 1.6.0.6 _______________________________________________ Anaconda-devel-list mailing list Anaconda-devel-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/anaconda-devel-list