[rhel6-branch] Fix the traceback from afbf164a1ce35aef50075cf0e22b9ece057bf1a1.

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

 



(Original commit message was: mpath: create /etc/multipath/bindings file.)

Related: rhbz#640735
---
 storage/__init__.py |   15 +++++++++------
 1 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/storage/__init__.py b/storage/__init__.py
index bc7af31..00053ce 100644
--- a/storage/__init__.py
+++ b/storage/__init__.py
@@ -2136,16 +2136,19 @@ class FSSet(object):
             open(mdadm_path, "w").write(mdadm_conf)
 
         # /etc/multipath.conf
-        multipath_path = os.path.normpath("%s/etc/multipath.conf" % instPath)
-        bindings_path = os.path.normpath("%s/etc/multipath/bindings" % instPath)
         multipath_conf = self.multipathConf()
-        conf_contents = multipath_conf.write()
-        bindings_contents = multipath_conf.write_bindings()
-        if conf_contents:
+        if multipath_conf:
+            multipath_path = os.path.normpath("%s/etc/multipath.conf" %
+                                              instPath)
+            bindings_path = os.path.normpath("%s/etc/multipath/bindings" %
+                                             instPath)
+            conf_contents = multipath_conf.write()
+            bindings_contents = multipath_conf.write_bindings()
             open(multipath_path, "w").write(conf_contents)
-        if bindings_contents:
             iutil.mkdirChain(os.path.dirname(bindings_path))
             open(bindings_path, "w").write(bindings_contents)
+        else:
+            log.info("not writing out mpath configuration")
 
     def crypttab(self):
         # if we are upgrading, do we want to update crypttab?
-- 
1.7.3.3

_______________________________________________
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