[master 3/3] iscsi, logging: reuse the global ISCSID in has_iscsi().

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

 



This prevents calling find_program_in_path() repeatedly and having the log
message "ISCSID is /sbin/iscsid" all over the place.
---
 pyanaconda/storage/iscsi.py |   23 +++++++++++------------
 1 files changed, 11 insertions(+), 12 deletions(-)

diff --git a/pyanaconda/storage/iscsi.py b/pyanaconda/storage/iscsi.py
index a29fc36..26fa5b5 100644
--- a/pyanaconda/storage/iscsi.py
+++ b/pyanaconda/storage/iscsi.py
@@ -45,19 +45,18 @@ INITIATOR_FILE="/etc/iscsi/initiatorname.iscsi"
 
 def has_iscsi():
     global ISCSID
-    location = iutil.find_program_in_path("iscsid")
-    if location:
-        ISCSID = location
-
-    if ISCSID == "" or not has_libiscsi:
-        return False
-
-    log.info("ISCSID is %s" % (ISCSID,))
-
-    # make sure the module is loaded
-    if not os.access("/sys/module/iscsi_tcp", os.X_OK):
-        return False
-    return True
+    
+    if os.access("/sys/module/iscsi_tcp", os.X_OK):
+        if len(ISCSID):
+            return True
+        else:
+            location = iutil.find_program_in_path("iscsid")
+            if location:
+                ISCSID = location
+                log.info("ISCSID is %s" % (ISCSID,))
+                return True
+
+    return False
 
 def randomIname():
     """Generate a random initiator name the same way as iscsi-iname"""
-- 
1.7.1.1

_______________________________________________
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