[PATCH 2/2] Some iscsi cleanups

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

 



-Improve iscsi error logging
-Use udev_settle from udev.py instead of calling udevadm ourselves
---
 storage/iscsi.py |   16 +++++++++-------
 1 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/storage/iscsi.py b/storage/iscsi.py
index e5e696a..5bf6d28 100644
--- a/storage/iscsi.py
+++ b/storage/iscsi.py
@@ -19,6 +19,7 @@
 #
 
 from constants import *
+from udev import *
 import os
 import iutil
 from flags import flags
@@ -84,9 +85,7 @@ def stabilize(intf = None):
     # It is possible when we get here the events for the new devices
     # are not send yet, so sleep to make sure the events are fired
     time.sleep(2)
-    iutil.execWithRedirect("udevadm", [ "settle" ],
-                           stdout = "/dev/tty5", stderr="/dev/tty5",
-                           searchPath = 1)
+    udev_settle()
     if intf:
         w.pop()
 
@@ -138,9 +137,9 @@ class iscsi(object):
                 node.login()
                 self.nodes.append(node)
                 self.ibftNodes.append(node)
-            except:
-                # FIXME, what to do when we cannot log in to a firmware
-                # provided node ??
+            except IOError, e:
+                log.error("Could not log into ibft iscsi target %s: %s" %
+                          (node.name, str(e)))
                 pass
 
         stabilize(intf)
@@ -228,7 +227,10 @@ class iscsi(object):
                 node.login()
                 self.nodes.append(node)
                 logged_in = logged_in + 1
-            except:
+            except IOError, e:
+                log.warning(
+                    _("Could not log into discovered iscsi target %s: %s" %
+                    (node.name, str(e)))
                 # some nodes may require different credentials
                 pass
 
-- 
1.6.5.2

_______________________________________________
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