Hello community,

here is the log from the commit of package yast2-iscsi-client for 
openSUSE:Factory checked in at 2015-07-16 17:17:38
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-iscsi-client (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-iscsi-client.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-iscsi-client"

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-iscsi-client/yast2-iscsi-client.changes    
2015-02-08 13:02:39.000000000 +0100
+++ 
/work/SRC/openSUSE:Factory/.yast2-iscsi-client.new/yast2-iscsi-client.changes   
    2015-07-16 17:17:39.000000000 +0200
@@ -1,0 +2,9 @@
+Thu Jun 18 14:22:05 CEST 2015 - [email protected]
+
+- warn only once about non-standard initiator name (bsc#926621)
+- improve help text concerning re-discovery of targets and 
+  start-up mode (bsc#924502, bsc#924691, bsc#924693) and about
+  authentication settings (bsc#924685)
+- 3.1.22
+
+-------------------------------------------------------------------

Old:
----
  yast2-iscsi-client-3.1.21.tar.bz2

New:
----
  yast2-iscsi-client-3.1.22.tar.bz2

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

Other differences:
------------------
++++++ yast2-iscsi-client.spec ++++++
--- /var/tmp/diff_new_pack.TeftUA/_old  2015-07-16 17:17:40.000000000 +0200
+++ /var/tmp/diff_new_pack.TeftUA/_new  2015-07-16 17:17:40.000000000 +0200
@@ -1,7 +1,7 @@
 #
 # spec file for package yast2-iscsi-client
 #
-# Copyright (c) 2015 SUSE LINUX Products GmbH, Nuernberg, Germany.
+# Copyright (c) 2015 SUSE LINUX GmbH, Nuernberg, Germany.
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-iscsi-client
-Version:        3.1.21
+Version:        3.1.22
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

++++++ yast2-iscsi-client-3.1.21.tar.bz2 -> yast2-iscsi-client-3.1.22.tar.bz2 
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-iscsi-client-3.1.21/CONTRIBUTING.md 
new/yast2-iscsi-client-3.1.22/CONTRIBUTING.md
--- old/yast2-iscsi-client-3.1.21/CONTRIBUTING.md       2015-02-06 
14:43:26.000000000 +0100
+++ new/yast2-iscsi-client-3.1.22/CONTRIBUTING.md       2015-07-01 
11:37:38.000000000 +0200
@@ -12,13 +12,13 @@
 -----------
 
 If you find a problem, please report it either using
-[Bugzilla](https://bugzilla.novell.com/enter_bug.cgi?format=guided&product=openSUSE+Factory&component=YaST2)
+[Bugzilla](https://bugzilla.suse.com/enter_bug.cgi?format=guided&product=openSUSE+Factory&component=YaST2)
 or [GitHub issues](../../issues). (For Bugzilla, use the [simplified
 
registration](https://secure-www.novell.com/selfreg/jsp/createSimpleAccount.jsp)
 if you don't have an account yet.)
 
 If you find a problem, please report it either using
-[Bugzilla](https://bugzilla.novell.com/) or GitHub issues. We can't guarantee
+[Bugzilla](https://bugzilla.suse.com/) or GitHub issues. We can't guarantee
 that every bug will be fixed, but we'll try.
 
 When creating a bug report, please follow our [bug reporting
@@ -71,7 +71,7 @@
 [widely used
 
conventions](http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html).
 
-If your commit is related to a bug in Buzgilla or an issue on GitHub, make sure
+If your commit is related to a bug in Bugzilla or an issue on GitHub, make sure
 you mention it in the commit message for cross-reference. Use format like
 bnc#775814 or gh#yast/yast-foo#42. See also [GitHub
 
autolinking](https://help.github.com/articles/github-flavored-markdown#references)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-iscsi-client-3.1.21/package/yast2-iscsi-client.changes 
new/yast2-iscsi-client-3.1.22/package/yast2-iscsi-client.changes
--- old/yast2-iscsi-client-3.1.21/package/yast2-iscsi-client.changes    
2015-02-06 14:43:26.000000000 +0100
+++ new/yast2-iscsi-client-3.1.22/package/yast2-iscsi-client.changes    
2015-07-01 11:37:38.000000000 +0200
@@ -1,4 +1,13 @@
 -------------------------------------------------------------------
+Thu Jun 18 14:22:05 CEST 2015 - [email protected]
+
+- warn only once about non-standard initiator name (bsc#926621)
+- improve help text concerning re-discovery of targets and 
+  start-up mode (bsc#924502, bsc#924691, bsc#924693) and about
+  authentication settings (bsc#924685)
+- 3.1.22
+
+-------------------------------------------------------------------
 Fri Feb  6 13:02:45 UTC 2015 - [email protected]
 
 - The unit tests are now compatible with RSpec 3 (bnc#916364)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-iscsi-client-3.1.21/package/yast2-iscsi-client.spec 
new/yast2-iscsi-client-3.1.22/package/yast2-iscsi-client.spec
--- old/yast2-iscsi-client-3.1.21/package/yast2-iscsi-client.spec       
2015-02-06 14:43:26.000000000 +0100
+++ new/yast2-iscsi-client-3.1.22/package/yast2-iscsi-client.spec       
2015-07-01 11:37:38.000000000 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-iscsi-client
-Version:        3.1.21
+Version:        3.1.22
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-iscsi-client-3.1.21/src/include/iscsi-client/helps.rb 
new/yast2-iscsi-client-3.1.22/src/include/iscsi-client/helps.rb
--- old/yast2-iscsi-client-3.1.21/src/include/iscsi-client/helps.rb     
2015-02-06 14:43:26.000000000 +0100
+++ new/yast2-iscsi-client-3.1.22/src/include/iscsi-client/helps.rb     
2015-07-01 11:37:38.000000000 +0200
@@ -88,9 +88,12 @@
           ),
         # table of connected targets
         "server_table"   => _(
-          "List of current sessions. To add a new target, select it and press 
<b>Add</b>.\n" +
-            "To remove it, press <b>Log Out</b>.\n" +
-            "To change the start-up status, press <b>Toggle</b>.\n"
+          "<p>List of current sessions.</p>" \
+            "<p>Use the <b>Add</b> button to get additional targets. A 
discovery is started to " \
+            "detect new targets and the start-up mode of already connected 
targets keeps " \
+            "unchanged.<br>" \
+            "Use <b>Log Out</b> to cancel the connection and with it remove 
the target from the list.<br>" \
+            "To change the start-up status, press <b>Edit</b>.</p>"
         ) +
           # Warning
           _("<h1>Warning</h1>") +
@@ -98,7 +101,7 @@
             "<p>When accessing an iSCSI device <b>READ</b>/<b>WRITE</b>, make 
sure that this access is exclusive. Otherwise there is a potential risk of data 
corruption.</p>\n"
           ),
         "initiator_name" => _(
-          "<p><b>InitiatorName</b> is a value from 
<tt>/etc/iscsi/initiatorname.iscsi</tt>. \nIn case you have iBFT, this value 
will be added from there and you are only able to change it in the BIOS 
setup.</p>"
+          "<p><b>Initiator Name</b> is a value from 
<tt>/etc/iscsi/initiatorname.iscsi</tt>. \nIn case you have iBFT, this value 
will be added from there and you are only able to change it in the BIOS 
setup.</p>"
         ),
         "isns"           => _(
           "If you want to use <b>iSNS</b> (Internet  Storage  Name Service) 
for discovering targets instead of the default SendTargets method,\nfill in the 
IP address of the iSNS server and port. The default port should be 3205.\n"
@@ -106,7 +109,7 @@
         # discovery new target
         "discovery"      => _("<h1>iSCSI Initiator</h1>") +
           _(
-            "Enter the <b>IP Address</b> of the discovered server.\n" +
+            "Enter the <b>IP Address</b> of the iSCSI target server.\n" +
               "Only change <b>Port</b> if needed. For authentication, use 
<b>Username</b> and <b>Password</b>. If you do not need authentication,\n" +
               "select <b>No Authentication</b>.\n"
           ) +
@@ -126,9 +129,6 @@
         "conn_auth"      => _(
           "<h1>iSCSI Initiator</h1>"
         ) +
-          _(
-            "Select the type of authentication and enter the <b>Username</b> 
and <b>Password</b>."
-          ) +
           _("<h1>Startup</h1>") +
           _(
             "<p><b>manual</b> is for iSCSI targets which are not to be 
connected by\n" +
@@ -137,10 +137,32 @@
               "root is on iSCSI. As such it will be evaluated by the 
initrd.</p>\n" +
               "<p><b>automatic</b> is for iSCSI targets to be connected when 
the iSCSI service\n" +
               "starts up.</p>\n"
+          ) +
+        _("<h1>Authentication</h1>") +
+        _(
+          "<p>The default setting here is <i>No Authentication</i>. Uncheck 
the checkbox if " \
+          "authentication is needed for security reasons. Enter 
<b>Username</b> and <b>Password</b> " \
+          "for 'Incoming Authentication', 'Outgoing Authentication' or for 
both.</p>" \
+          "<p><b>Please note:</b><br>" \
+          "'Incoming Authentication' here correlates to 'Outgoing 
Authentication' "\
+          "on iSCSI target server side and the other way round.</p>"
           ),
         # list of discovered targets
         "discovered"     => _(
-          "List of discovered targets. Start a new <b>Discovery</b> or 
<b>Connect</b> to any target."
+          "<p>This screen shows the list of discovered targets.</p>" \
+          "<p>Use the <b>Discovery</b> button to get available iSCSI targets " 
\
+          "from a server specified by IP address.<br>" \
+          "<b>Log In</b> to a target to establih the connection. If login was 
successful " \
+          "the column <i>Connected</i> shows status 'True' and the target will 
appear on " \
+          "the <i>Connected Targets</i> screen.<br>" \
+          "To remove a target use the <b>Delete</b> button.<br> <b>Hint:</b> " 
\
+          "Removing of targets is only possible for not connected onces. " \
+          "If required, <b>Log Out</b> at <i>Connected Targets</i> first.</p>" 
\
+          "<p><b>Please note:</b> Starting the <b>Discovery</b> again means 
doing a re-discovery " \
+          "of targets which possibly will change the start-up mode of already 
connected targets " \
+          "(to default 'manual'). " \
+          "Switch to <i>Connected Targets</i> screen and use the <b>Add</b> 
button if you want " \
+          "to add new targets without changing the start-up mode.</p>"
         ),
         "ibft_table"     => _("<h1>iBTF</h1>") +
           "The <p>iSCSI Boot Firmware Table</p> is a table created by the 
iSCSI boot firmware in order to\npass parameters about the iSCSI boot device to 
the loaded OS."
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-iscsi-client-3.1.21/src/include/iscsi-client/widgets.rb 
new/yast2-iscsi-client-3.1.22/src/include/iscsi-client/widgets.rb
--- old/yast2-iscsi-client-3.1.21/src/include/iscsi-client/widgets.rb   
2015-02-06 14:43:26.000000000 +0100
+++ new/yast2-iscsi-client-3.1.22/src/include/iscsi-client/widgets.rb   
2015-07-01 11:37:38.000000000 +0200
@@ -407,6 +407,12 @@
       #  is freely selectable. For further details please check the iSCSI spec.
 
       i_name = Convert.to_string(UI.QueryWidget(:initiator_name, :Value))
+
+      # name not changed at all or already saved after checking it
+      if IscsiClientLib.initiatorname == i_name
+        return true
+      end
+
       # regexp for "yyyy-mm."
       reg1 = "[[:digit:]]{4}-[[:digit:]]{2}."
       # regexp for "cz.suse" or just "suse", "cz.su-se"
@@ -419,18 +425,20 @@
         Builtins.regexpmatch(i_name, Builtins.sformat("^eui.%1%2$", reg1, 
reg2))
 
       if !correct
-        continue = Popup.ContinueCancel(
+        go_on = Popup.YesNoHeadline( _("Incorrect Initiator Name"),
           _(
-            "Incorrect InitiatorName.\n" +
-              "The correct syntax is\n" +
-              "iqn.yyyy-mm.reversed.domain.name[:identifier]\n" +
-              "or eui.yyyy-mm.reversed.domain.name[:identifier]\n" +
-              "\n" +
-              "Example:\n" +
-              "iqn.2007-04.cz.server:storage.disk.sdb\n"
+            "\n" +
+            "The correct syntax is\n" +
+            "iqn.yyyy-mm.reversed.domain.name[:identifier]\n" +
+            "or eui.yyyy-mm.reversed.domain.name[:identifier]\n" +
+            "\n" +
+            "Example:\n" +
+            "iqn.2007-04.cz.server:storage.disk.sdb\n" +
+            "\n" +
+            "Do you want to use the name?\n"
           )
         )
-        return continue
+        return go_on
       else
         return true
       end


Reply via email to