Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package multipath-tools for openSUSE:Factory checked in at 2021-04-10 15:25:53 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/multipath-tools (Old) and /work/SRC/openSUSE:Factory/.multipath-tools.new.2401 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "multipath-tools" Sat Apr 10 15:25:53 2021 rev:130 rq:882546 version:0.8.5+30+suse.633836e Changes: -------- --- /work/SRC/openSUSE:Factory/multipath-tools/multipath-tools.changes 2021-03-21 23:19:24.112719290 +0100 +++ /work/SRC/openSUSE:Factory/.multipath-tools.new.2401/multipath-tools.changes 2021-04-10 15:25:56.550277186 +0200 @@ -1,0 +2,7 @@ +Thu Apr 01 11:01:34 UTC 2021 - mwi...@suse.com + +- Update to version 0.8.5+30+suse.633836e: + * multipathd: give up "add missing path" after multiple failures + (bsc#1183963) + +------------------------------------------------------------------- Old: ---- multipath-tools-0.8.5+29+suse.5dabcd2.obscpio New: ---- multipath-tools-0.8.5+30+suse.633836e.obscpio ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ multipath-tools.spec ++++++ --- /var/tmp/diff_new_pack.FSb0zw/_old 2021-04-10 15:25:58.118279030 +0200 +++ /var/tmp/diff_new_pack.FSb0zw/_new 2021-04-10 15:25:58.122279035 +0200 @@ -39,7 +39,7 @@ %define _sysdir usr/lib Name: multipath-tools -Version: 0.8.5+29+suse.5dabcd2 +Version: 0.8.5+30+suse.633836e Release: 0 Summary: Tools to Manage Multipathed Devices with the device-mapper License: GPL-2.0-only ++++++ _servicedata ++++++ --- /var/tmp/diff_new_pack.FSb0zw/_old 2021-04-10 15:25:58.162279082 +0200 +++ /var/tmp/diff_new_pack.FSb0zw/_new 2021-04-10 15:25:58.162279082 +0200 @@ -1,4 +1,4 @@ <servicedata> <service name="tar_scm"> <param name="url">https://github.com/openSUSE/multipath-tools.git</param> - <param name="changesrevision">5dabcd21663219ae45e694d9f67410978e1b7c94</param></service></servicedata> \ No newline at end of file + <param name="changesrevision">633836e481465d042d7707b498678d3cf8216e2e</param></service></servicedata> \ No newline at end of file ++++++ multipath-tools-0.8.5+29+suse.5dabcd2.obscpio -> multipath-tools-0.8.5+30+suse.633836e.obscpio ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/multipath-tools-0.8.5+29+suse.5dabcd2/libmultipath/discovery.c new/multipath-tools-0.8.5+30+suse.633836e/libmultipath/discovery.c --- old/multipath-tools-0.8.5+29+suse.5dabcd2/libmultipath/discovery.c 2021-03-18 10:08:34.000000000 +0100 +++ new/multipath-tools-0.8.5+30+suse.633836e/libmultipath/discovery.c 2021-03-29 22:34:52.000000000 +0200 @@ -2249,6 +2249,22 @@ if (pp->initialized != INIT_FAILED) { pp->initialized = INIT_MISSING_UDEV; pp->tick = conf->retrigger_delay; + } else if (pp->retriggers >= conf->retrigger_tries && + (pp->state == PATH_UP || pp->state == PATH_GHOST)) { + /* + * We have failed to read udev info for this path + * repeatedly. We used the fallback in get_uid() + * if there was any, and still got no WWID, + * although the path is allegedly up. + * It's likely that this path is not fit for + * multipath use. + */ + char buf[16]; + + snprint_path(buf, sizeof(buf), "%T", pp, 0); + condlog(1, "%s: no WWID in state \"%s\", giving up", + pp->dev, buf); + return PATHINFO_SKIPPED; } return PATHINFO_OK; } diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/multipath-tools-0.8.5+29+suse.5dabcd2/multipathd/main.c new/multipath-tools-0.8.5+30+suse.633836e/multipathd/main.c --- old/multipath-tools-0.8.5+29+suse.5dabcd2/multipathd/main.c 2021-03-18 10:08:34.000000000 +0100 +++ new/multipath-tools-0.8.5+30+suse.633836e/multipathd/main.c 2021-03-29 22:34:52.000000000 +0200 @@ -2182,13 +2182,13 @@ ev_add_path(pp, vecs, 1); pp->tick = 1; } else { + if (ret == PATHINFO_SKIPPED) + return -1; /* * We failed multiple times to initialize this * path properly. Don't re-check too often. */ pp->checkint = max_checkint; - if (ret == PATHINFO_SKIPPED) - return -1; } } return 0; @@ -2455,6 +2455,8 @@ vector_foreach_slot (vecs->pathvec, pp, i) { rc = check_path(vecs, pp, ticks); if (rc < 0) { + condlog(1, "%s: check_path() failed, removing", + pp->dev); vector_del_slot(vecs->pathvec, i); free_path(pp); i--; ++++++ multipath-tools.obsinfo ++++++ --- /var/tmp/diff_new_pack.FSb0zw/_old 2021-04-10 15:25:58.430279397 +0200 +++ /var/tmp/diff_new_pack.FSb0zw/_new 2021-04-10 15:25:58.434279402 +0200 @@ -1,5 +1,5 @@ name: multipath-tools -version: 0.8.5+29+suse.5dabcd2 -mtime: 1616058514 -commit: 5dabcd21663219ae45e694d9f67410978e1b7c94 +version: 0.8.5+30+suse.633836e +mtime: 1617050092 +commit: 633836e481465d042d7707b498678d3cf8216e2e