Re: [PATCH 1/4] We no longer use iscsiadm anywhere

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

 



looks good.
On Tue, Mar 17, 2009 at 10:22:52PM +0100, Hans de Goede wrote:
> Hurray!
> ---
>  storage/iscsi.py |   11 +----------
>  1 files changed, 1 insertions(+), 10 deletions(-)
> 
> diff --git a/storage/iscsi.py b/storage/iscsi.py
> index 1add6ee..11d3656 100644
> --- a/storage/iscsi.py
> +++ b/storage/iscsi.py
> @@ -46,8 +46,6 @@ except:
>  # Note that stage2 copies all files under /sbin to /usr/sbin
>  global ISCSID
>  ISCSID=""
> -global ISCSIADM
> -ISCSIADM = ""
>  INITIATOR_FILE="/etc/iscsi/initiatorname.iscsi"
>  
>  def find_iscsi_files():
> @@ -57,20 +55,13 @@ def find_iscsi_files():
>              path="%s/iscsid" % (dir,)
>              if os.access(path, os.X_OK):
>                  ISCSID=path
> -    global ISCSIADM
> -    if ISCSIADM == "":
> -        for dir in ("/usr/sbin", "/tmp/updates", "/mnt/source/RHupdates"):
> -            path="%s/iscsiadm" % (dir,)
> -            if os.access(path, os.X_OK):
> -                ISCSIADM=path
>  
>  def has_iscsi():
>      find_iscsi_files()
> -    if ISCSID == "" or ISCSIADM == "" or not has_libiscsi:
> +    if ISCSID == "" or not has_libiscsi:
>          return False
>  
>      log.info("ISCSID is %s" % (ISCSID,))
> -    log.info("ISCSIADM is %s" % (ISCSIADM,))
>  
>      # make sure the module is loaded
>      if not os.access("/sys/module/iscsi_tcp", os.X_OK):
> -- 
> 1.6.1.3
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

-- 
Joel Andres Granados
Brno, Czech Republic, Red Hat.

_______________________________________________
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