Hello community, here is the log from the commit of package yast2-iscsi-client for openSUSE:Factory checked in at 2019-03-06 19:02:28 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/yast2-iscsi-client (Old) and /work/SRC/openSUSE:Factory/.yast2-iscsi-client.new.28833 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "yast2-iscsi-client" Wed Mar 6 19:02:28 2019 rev:122 rq:681465 version:4.1.6 Changes: -------- --- /work/SRC/openSUSE:Factory/yast2-iscsi-client/yast2-iscsi-client.changes 2019-03-04 09:15:54.656649182 +0100 +++ /work/SRC/openSUSE:Factory/.yast2-iscsi-client.new.28833/yast2-iscsi-client.changes 2019-03-06 19:02:32.145088595 +0100 @@ -1,0 +2,6 @@ +Mon Mar 4 13:36:13 UTC 2019 - jsr...@suse.cz + +- fixed parsing of iscsiadm -m node -P 1 (bsc#1121806) +- 4.1.6 + +------------------------------------------------------------------- Old: ---- yast2-iscsi-client-4.1.5.tar.bz2 New: ---- yast2-iscsi-client-4.1.6.tar.bz2 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ yast2-iscsi-client.spec ++++++ --- /var/tmp/diff_new_pack.lHWXUg/_old 2019-03-06 19:02:32.813088415 +0100 +++ /var/tmp/diff_new_pack.lHWXUg/_new 2019-03-06 19:02:32.817088414 +0100 @@ -17,7 +17,7 @@ Name: yast2-iscsi-client -Version: 4.1.5 +Version: 4.1.6 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build ++++++ yast2-iscsi-client-4.1.5.tar.bz2 -> yast2-iscsi-client-4.1.6.tar.bz2 ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-iscsi-client-4.1.5/package/yast2-iscsi-client.changes new/yast2-iscsi-client-4.1.6/package/yast2-iscsi-client.changes --- old/yast2-iscsi-client-4.1.5/package/yast2-iscsi-client.changes 2019-03-01 15:47:03.000000000 +0100 +++ new/yast2-iscsi-client-4.1.6/package/yast2-iscsi-client.changes 2019-03-04 15:12:49.000000000 +0100 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Mon Mar 4 13:36:13 UTC 2019 - jsr...@suse.cz + +- fixed parsing of iscsiadm -m node -P 1 (bsc#1121806) +- 4.1.6 + +------------------------------------------------------------------- Fri Mar 1 13:36:35 UTC 2019 - snw...@suse.com - fix iBFT handling (bsc#1119698) diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-iscsi-client-4.1.5/package/yast2-iscsi-client.spec new/yast2-iscsi-client-4.1.6/package/yast2-iscsi-client.spec --- old/yast2-iscsi-client-4.1.5/package/yast2-iscsi-client.spec 2019-03-01 15:47:03.000000000 +0100 +++ new/yast2-iscsi-client-4.1.6/package/yast2-iscsi-client.spec 2019-03-04 15:12:49.000000000 +0100 @@ -17,7 +17,7 @@ Name: yast2-iscsi-client -Version: 4.1.5 +Version: 4.1.6 Release: 0 BuildRoot: %{_tmppath}/%{name}-%{version}-build diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-iscsi-client-4.1.5/src/modules/IscsiClientLib.rb new/yast2-iscsi-client-4.1.6/src/modules/IscsiClientLib.rb --- old/yast2-iscsi-client-4.1.5/src/modules/IscsiClientLib.rb 2019-03-01 15:47:03.000000000 +0100 +++ new/yast2-iscsi-client-4.1.6/src/modules/IscsiClientLib.rb 2019-03-04 15:12:49.000000000 +0100 @@ -473,12 +473,23 @@ target = "" portal = "" iface = "" + dumped = true Builtins.y2milestone("Got data: %1", data) + # Each entry starts with Target:, the other two values are optional + # (except the first entry) and, if missing, are inherited from previous + # entry. Therefore: Dump whatever is cached on Target: entry and once + # again at the end. Example input in the test case. + Builtins.foreach(data) do |row| row = Builtins.substring(row, Builtins.findfirstnotof(row, "\t "), 999) if Builtins.search(row, "Target:") != nil + if !dumped + # don't add Scope:Link IPv6 address + ret << "#{portal} #{target} #{iface}" if !portal.start_with?("[fe80:") + end target = Ops.get(Builtins.splitstring(row, " "), 1, "") + dumped = false elsif Builtins.search(row, "Portal:") != nil if Builtins.search(row, "Current Portal:") != nil portal = Ops.get(Builtins.splitstring(row, " "), 2, "") @@ -494,12 +505,13 @@ elsif Builtins.search(row, "Iface Name:") != nil iface = Ops.get(Builtins.splitstring(row, " "), 2, "") iface = Ops.get(@iface_file, iface, iface) - # don't add Scope:Link IPv6 address - if !portal.start_with?("[fe80:") - ret = ret << "#{portal} #{target} #{iface}" - end end end + if !dumped + # don't add Scope:Link IPv6 address + ret << "#{portal} #{target} #{iface}" if !portal.start_with?("[fe80:") + end + Builtins.y2milestone("ScanDiscovered ret:%1", ret) deep_copy(ret) end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/yast2-iscsi-client-4.1.5/test/ScanDiscovered_spec.rb new/yast2-iscsi-client-4.1.6/test/ScanDiscovered_spec.rb --- old/yast2-iscsi-client-4.1.5/test/ScanDiscovered_spec.rb 2019-03-01 15:47:03.000000000 +0100 +++ new/yast2-iscsi-client-4.1.6/test/ScanDiscovered_spec.rb 2019-03-04 15:12:49.000000000 +0100 @@ -58,12 +58,14 @@ ["Target: iqn.2013-10.de.suse:test_file2", "\tPortal: [fe80::a00:27ff:fe1b:a7fe]:3260,1", "\t\tIface Name: default", + "Target: iqn.2013-10.de.suse:test_file2", "\tPortal: [2620:113:80c0:8080:e051:f9ea:73c7:9171]:3260,1", "\t\tIface Name: default", + "Target: iqn.2013-10.de.suse:test_file2", "\tPortal: 10.120.66.182:3260,1", - "\t\tIface Name: default", - "\tPortal: [2620:113:80c0:8080:a00:27ff:fe1b:a7fe]:3260,1", - "\t\tIface Name: default"] + "Target: iqn.2013-10.de.suse:test_file2", + "\tPortal: [2620:113:80c0:8080:a00:27ff:fe1b:a7fe]:3260,1"] + )). to eq( [ "[2620:113:80c0:8080:e051:f9ea:73c7:9171]:3260 iqn.2013-10.de.suse:test_file2 default",