Hello community,

here is the log from the commit of package yast2-iscsi-lio-server for 
openSUSE:Factory checked in at 2014-07-28 16:21:32
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-iscsi-lio-server (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-iscsi-lio-server.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-iscsi-lio-server"

Changes:
--------
--- 
/work/SRC/openSUSE:Factory/yast2-iscsi-lio-server/yast2-iscsi-lio-server.changes
    2014-05-16 18:18:34.000000000 +0200
+++ 
/work/SRC/openSUSE:Factory/.yast2-iscsi-lio-server.new/yast2-iscsi-lio-server.changes
       2014-07-28 16:21:33.000000000 +0200
@@ -1,0 +2,7 @@
+Thu Jul 24 11:59:18 CEST 2014 - [email protected]
+
+- bind all IP addresses to a target if requested by user 
+  (bnc #885688)
+- 3.1.8 
+
+-------------------------------------------------------------------

Old:
----
  yast2-iscsi-lio-server-3.1.7.tar.bz2

New:
----
  yast2-iscsi-lio-server-3.1.8.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-iscsi-lio-server.spec ++++++
--- /var/tmp/diff_new_pack.ivcMqf/_old  2014-07-28 16:21:34.000000000 +0200
+++ /var/tmp/diff_new_pack.ivcMqf/_new  2014-07-28 16:21:34.000000000 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-iscsi-lio-server
-Version:        3.1.7
+Version:        3.1.8
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

++++++ yast2-iscsi-lio-server-3.1.7.tar.bz2 -> 
yast2-iscsi-lio-server-3.1.8.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-iscsi-lio-server-3.1.7/package/yast2-iscsi-lio-server.changes 
new/yast2-iscsi-lio-server-3.1.8/package/yast2-iscsi-lio-server.changes
--- old/yast2-iscsi-lio-server-3.1.7/package/yast2-iscsi-lio-server.changes     
2014-05-15 12:10:47.000000000 +0200
+++ new/yast2-iscsi-lio-server-3.1.8/package/yast2-iscsi-lio-server.changes     
2014-07-28 13:30:42.000000000 +0200
@@ -1,4 +1,11 @@
 -------------------------------------------------------------------
+Thu Jul 24 11:59:18 CEST 2014 - [email protected]
+
+- bind all IP addresses to a target if requested by user 
+  (bnc #885688)
+- 3.1.8 
+
+-------------------------------------------------------------------
 Thu May 15 09:53:44 CEST 2014 - [email protected]
 
 - save configuration on Finish (call lio_dump and tcm_dump) 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-iscsi-lio-server-3.1.7/package/yast2-iscsi-lio-server.spec 
new/yast2-iscsi-lio-server-3.1.8/package/yast2-iscsi-lio-server.spec
--- old/yast2-iscsi-lio-server-3.1.7/package/yast2-iscsi-lio-server.spec        
2014-05-15 12:10:47.000000000 +0200
+++ new/yast2-iscsi-lio-server-3.1.8/package/yast2-iscsi-lio-server.spec        
2014-07-28 13:30:42.000000000 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-iscsi-lio-server
-Version:        3.1.7
+Version:        3.1.8
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-iscsi-lio-server-3.1.7/src/include/iscsi-lio-server/dialogs.rb 
new/yast2-iscsi-lio-server-3.1.8/src/include/iscsi-lio-server/dialogs.rb
--- old/yast2-iscsi-lio-server-3.1.7/src/include/iscsi-lio-server/dialogs.rb    
2014-05-15 12:10:47.000000000 +0200
+++ new/yast2-iscsi-lio-server-3.1.8/src/include/iscsi-lio-server/dialogs.rb    
2014-07-28 13:30:42.000000000 +0200
@@ -139,6 +139,8 @@
                   ),
                   HWeight(1, InputField(Id(:port), _("Port number"), "3260"))
                 ),
+                VSpacing(0.2),
+                Left(HBox(CheckBox(Id(:add_ip), _("Bind all IP addresses"), 
false))),
                 VSpacing(0.5),
                 Left(HBox(CheckBox(Id(:auth), _("Use Authentication"), true))),
                 VSpacing(0.5),
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-iscsi-lio-server-3.1.7/src/include/iscsi-lio-server/widgets.rb 
new/yast2-iscsi-lio-server-3.1.8/src/include/iscsi-lio-server/widgets.rb
--- old/yast2-iscsi-lio-server-3.1.7/src/include/iscsi-lio-server/widgets.rb    
2014-05-15 12:10:47.000000000 +0200
+++ new/yast2-iscsi-lio-server-3.1.8/src/include/iscsi-lio-server/widgets.rb    
2014-07-28 13:30:42.000000000 +0200
@@ -1013,14 +1013,14 @@
       )
       ip = Convert.to_string(UI.QueryWidget(:ipaddr, :Value))
       port = Builtins.tointeger(UI.QueryWidget(:port, :Value))
+      add_all = Convert.to_boolean(UI.QueryWidget(:add_ip, :Value))
+
       Builtins.y2milestone("storeModify ip:%1 port:%2 ipp:%3", ip, port, ipp)
-      if IP.Check6(ip)
-         ip = "[#{ip}]" # brackets needed around IPv6
-      end
-      np = Builtins.sformat("%1:%2", ip, port)
-      if !Builtins.isempty(ip) && np != ipp
+
+      np = IscsiLioData.FormatIpPort(ip, port)
+      if (!ip.empty? && np != ipp) || add_all
         chg = true
-        if !IscsiLioData.SetNetworkPortal(@curr_target, @curr_tpg, np)
+        if !IscsiLioData.SetNetworkPortal(@curr_target, @curr_tpg, np, port, 
add_all)
           txt = Builtins.sformat(_("Problem setting network portal to %1"), np)
           Popup.Error(txt)
         end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-iscsi-lio-server-3.1.7/src/modules/IscsiLioData.rb 
new/yast2-iscsi-lio-server-3.1.8/src/modules/IscsiLioData.rb
--- old/yast2-iscsi-lio-server-3.1.7/src/modules/IscsiLioData.rb        
2014-05-15 12:10:47.000000000 +0200
+++ new/yast2-iscsi-lio-server-3.1.8/src/modules/IscsiLioData.rb        
2014-07-28 13:30:42.000000000 +0200
@@ -369,15 +369,21 @@
       deep_copy(ret)
     end
 
+    # format ip and port information for use in commands
+    def FormatIpPort(ip, port)
+      # brackets needed around IPv6
+      ip = "[#{ip}]" if IP.Check6(ip)
+      return "#{ip}:#{port}"
+    end
+
     def GetNetworkPortal(tgt, tpg)
       Builtins.y2milestone("Data: %1, tgt: %2, tpg: %3", @data, tgt, tpg)
       ret = Builtins.maplist(
         Ops.get_list(@data, ["tgt", tgt, tpg, "ep", "np"], [])
       ) do |n|
         ip = n["ip"] || ""
-        ip = "[#{ip}]" if IP.Check6(ip)
-        port = n["port"] || 1
-        ipp = Builtins.sformat("%1:%2", ip, port )
+        port = n["port"] || 3260
+        ipp = FormatIpPort(ip, port)
       end
       deep_copy(ret)
     end
@@ -399,28 +405,33 @@
       deep_copy(ret)
     end
 
-    def SetNetworkPortal(tgt, tpg, np)
+    def SetNetworkPortal(tgt, tpg, np, new_port, add_all)
       Builtins.y2milestone("SetNetworkPortal tgt:%1 tpg:%2 np:%3", tgt, tpg, 
np)
-      kt = Ops.add(Ops.add(Ops.add(tgt, " "), tpg), " ")
-      if !Builtins.isempty(
-          Ops.get_list(@data, ["tgt", tgt, tpg, "ep", "np"], [])
-        )
-        LogExecCmd(
-          Ops.add(
-            Ops.add("lio_node --delnp ", kt),
-            Builtins.sformat(
-              "%1:%2",
-              Ops.get_string(@data, ["tgt", tgt, tpg, "ep", "np", 0, "ip"], 
""),
-              Ops.get_integer(
-                @data,
-                ["tgt", tgt, tpg, "ep", "np", 0, "port"],
-                1
-              )
-            )
-          )
-        )
+
+      target_info = "#{tgt} #{tpg}"
+      ip_list = Ops.get_list(@data, ["tgt", tgt, tpg, "ep", "np"], [])
+
+      if !ip_list.empty? && !add_all
+        ip_list.each do |ipp|
+          ip = ipp["ip"]
+          port = ipp["port"] || 3260
+          if ip
+            LogExecCmd("lio_node --delnp #{target_info} #{FormatIpPort(ip, 
port)}")
+          else
+            Builtins.y2error("IP address missing in data #{ipp}")
+          end
+        end
+      end
+
+      if add_all
+        ret = true
+        IscsiLioData.GetIpAddr.each do |ip|
+          success = LogExecCmd("lio_node --addnp #{target_info} 
#{FormatIpPort(ip, new_port)}")
+          ret = false if !success 
+        end
+      else
+        ret = LogExecCmd("lio_node --addnp #{target_info} #{np}")
       end
-      ret = LogExecCmd(Ops.add(Ops.add("lio_node --addnp ", kt), np))
       ret
     end
 
@@ -1429,7 +1440,7 @@
     publish :function => :GetTpgAuth, :type => "boolean (string, integer)"
     publish :function => :GetClntList, :type => "list <string> (string, 
integer)"
     publish :function => :GetClntLun, :type => "map <integer, integer> 
(string, integer, string)"
-    publish :function => :SetNetworkPortal, :type => "boolean (string, 
integer, string)"
+    publish :function => :SetNetworkPortal, :type => "boolean (string, 
integer, string, string, boolean)"
     publish :function => :GetTargets, :type => "list <list> ()"
     publish :function => :GetExportLun, :type => "string (integer, map)"
     publish :function => :GetExportAuth, :type => "list <map <string, any>> 
(string, integer)"

-- 
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to