Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package wicked for openSUSE:Factory checked 
in at 2024-04-10 17:48:52
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/wicked (Old)
 and      /work/SRC/openSUSE:Factory/.wicked.new.29460 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "wicked"

Wed Apr 10 17:48:52 2024 rev:100 rq:1166514 version:0.6.74

Changes:
--------
--- /work/SRC/openSUSE:Factory/wicked/wicked.changes    2024-03-21 
17:00:18.912088559 +0100
+++ /work/SRC/openSUSE:Factory/.wicked.new.29460/wicked.changes 2024-04-10 
17:49:09.045736406 +0200
@@ -1,0 +2,6 @@
+Tue Apr  9 15:39:01 UTC 2024 - Clemens Famulla-Conrad <cfamullacon...@suse.com>
+
+- client: do not convert sec to msec twice (bsc#1222105)
+  [+ 0005-client-do-to-not-convert-sec-to-msec-twice-bsc-1222105.patch] 
+
+-------------------------------------------------------------------

New:
----
  0005-client-do-to-not-convert-sec-to-msec-twice-bsc-1222105.patch

BETA DEBUG BEGIN:
  New:- client: do not convert sec to msec twice (bsc#1222105)
  [+ 0005-client-do-to-not-convert-sec-to-msec-twice-bsc-1222105.patch] 
BETA DEBUG END:

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

Other differences:
------------------
++++++ wicked.spec ++++++
--- /var/tmp/diff_new_pack.4v76aS/_old  2024-04-10 17:49:09.849765995 +0200
+++ /var/tmp/diff_new_pack.4v76aS/_new  2024-04-10 17:49:09.849765995 +0200
@@ -30,6 +30,7 @@
 Patch2:         0002-extensions-nbft-replace-nvme-show-nbft-with-nvme-nbf.patch
 Patch3:         0003-move-all-attribute-definitions-to-compiler-h.patch
 Patch4:         0004-hide-secrets-in-debug-log-bsc-1221194.patch
+Patch5:         
0005-client-do-to-not-convert-sec-to-msec-twice-bsc-1222105.patch
 #
 # Upstream First - openSUSE Build Service Policy:
 #

++++++ 0005-client-do-to-not-convert-sec-to-msec-twice-bsc-1222105.patch ++++++
>From 342a2dc7927f772dda50cd1db3f1b5ddbd96bc0b Mon Sep 17 00:00:00 2001
From: Marius Tomaschewski <m...@suse.com>
Date: Thu, 28 Mar 2024 16:06:18 +0100
Subject: [PATCH 1/3] fsm: fix to set extra waittime from xml in seconds

The extra waittime calculated from bridge stp=on
config timings is in seconds, do not convert to
msec timeout yet.
---
 src/fsm.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/fsm.c b/src/fsm.c
index caadd9b9..4d9922ea 100644
--- a/src/fsm.c
+++ b/src/fsm.c
@@ -2058,7 +2058,7 @@ ni_ifworker_set_config_origin(ni_ifworker_t *w, const 
char *new_origin)
 static void
 ni_ifworker_extra_waittime_from_xml(ni_ifworker_t *w)
 {
-       unsigned int extra_timeout = 0;
+       unsigned int extra_waittime = 0;
        const xml_node_t *brnode;
 
        if (!w || xml_node_is_empty(w->config.node))
@@ -2066,9 +2066,9 @@ ni_ifworker_extra_waittime_from_xml(ni_ifworker_t *w)
 
        /* Adding bridge dependent values (STP, Forwarding times) */
        if ((brnode = xml_node_get_child(w->config.node, "bridge")))
-               extra_timeout += ni_bridge_waittime_from_xml(brnode);
+               extra_waittime += ni_bridge_waittime_from_xml(brnode);
 
-       w->extra_waittime = (extra_timeout*1000);
+       w->extra_waittime = extra_waittime;
 }
 
 ni_iftype_t
-- 
2.35.3


>From 5bc4ec6fc30208c57cec98327f4c0b5a87774c3d Mon Sep 17 00:00:00 2001
From: Marius Tomaschewski <m...@suse.com>
Date: Thu, 28 Mar 2024 16:07:28 +0100
Subject: [PATCH 2/3] fsm: apply extra waittime to timeout additionally

---
 src/fsm.c | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/src/fsm.c b/src/fsm.c
index 4d9922ea..890f45e6 100644
--- a/src/fsm.c
+++ b/src/fsm.c
@@ -6119,6 +6119,7 @@ done:
 ni_timeout_t
 ni_fsm_find_max_timeout(ni_fsm_t *fsm, ni_timeout_t timeout)
 {
+       ni_timeout_t max = timeout;
        unsigned int i;
 
        if (!fsm || timeout >= NI_IFWORKER_INFINITE_TIMEOUT)
@@ -6126,19 +6127,13 @@ ni_fsm_find_max_timeout(ni_fsm_t *fsm, ni_timeout_t 
timeout)
 
        for (i = 0; i < fsm->workers.count; i++) {
                ni_ifworker_t *w = fsm->workers.data[i];
-               ni_timeout_t max, add;
+               ni_timeout_t add;
 
-               add = min_t(ni_timeout_t,
-                               NI_TIMEOUT_FROM_SEC(w->extra_waittime),
-                               NI_IFWORKER_INFINITE_TIMEOUT);
-               max = max_t(ni_timeout_t, timeout,
-                               fsm->worker_timeout + add);
-
-               timeout = min_t(ni_timeout_t, max,
-                               NI_IFWORKER_INFINITE_TIMEOUT);
+               add = NI_TIMEOUT_FROM_SEC(w->extra_waittime);
+               max = max_t(ni_timeout_t, max, timeout + add);
        }
 
-       return timeout;
+       return min_t(ni_timeout_t, max, NI_IFWORKER_INFINITE_TIMEOUT);
 }
 
 void
-- 
2.35.3


>From 31a074bcab2bb830ef7b073f9c023f5cb13c8bb4 Mon Sep 17 00:00:00 2001
From: Marius Tomaschewski <m...@suse.com>
Date: Thu, 28 Mar 2024 16:11:10 +0100
Subject: [PATCH 3/3] ifdown: fix to properly init second timeout

Don't init the --timeout seconds with worker default
timeout constant, which is in msec already.
Also, skip adding bridge stp=on interface setup time
as there is none when we're shutting down interfaces.
---
 client/ifdown.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/client/ifdown.c b/client/ifdown.c
index b80a3a0d..b7b2b834 100644
--- a/client/ifdown.c
+++ b/client/ifdown.c
@@ -123,7 +123,7 @@ ni_do_ifdown(int argc, char **argv)
        ni_ifworker_array_t ifmarked = NI_IFWORKER_ARRAY_INIT;
        ni_string_array_t ifnames = NI_STRING_ARRAY_INIT;
        unsigned int nmarked, max_state = NI_FSM_STATE_DEVICE_DOWN;
-       unsigned int seconds = NI_IFWORKER_DEFAULT_TIMEOUT;
+       unsigned int seconds = 0;
        ni_stringbuf_t sb = NI_STRINGBUF_INIT_DYNAMIC;
        ni_tristate_t opt_release = NI_TRISTATE_DEFAULT;
        ni_fsm_t *fsm;
@@ -223,9 +223,10 @@ usage:
        ifmarker.target_range.min = NI_FSM_STATE_NONE;
        ifmarker.target_range.max = max_state;
 
-       fsm->worker_timeout = ni_fsm_find_max_timeout(fsm,
-                                       NI_TIMEOUT_FROM_SEC(seconds));
-
+       if (seconds)
+               fsm->worker_timeout = NI_TIMEOUT_FROM_SEC(seconds);
+       else
+               fsm->worker_timeout = NI_IFWORKER_DEFAULT_TIMEOUT;
        if (fsm->worker_timeout == NI_IFWORKER_INFINITE_TIMEOUT)
                ni_debug_application("wait for interfaces infinitely");
        else
-- 
2.35.3

Reply via email to