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