autobuild[sn-devel-144]: intermittent test failure detected

[autobuild]

The autobuild test system (on sn-devel-144) has detected an intermittent 
failing test in 
the current master tree.

The autobuild log of the failure is available here:

   http://git.samba.org/autobuild.flakey.sn-devel-144/2017-01-26-0617/flakey.log

The samba build logs are available here:

   
http://git.samba.org/autobuild.flakey.sn-devel-144/2017-01-26-0617/samba.stderr
   
http://git.samba.org/autobuild.flakey.sn-devel-144/2017-01-26-0617/samba.stdout
  
The top commit at the time of the failure was:

commit 2cf141ed45b4f7b7754cb9525d987ff38495d789
Author: Uri Simchoni 
Date:   Thu Jan 19 07:46:57 2017 +0200

waf: backport finding of pkg-config

Allow the builder to customize the location of pkg-config
utility by setting PKGCONFIG environment variable.

This is backported from upstream waf.

Thanks to Zentaro Kavanagh  for
pointing that out and proposing the fix.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12529

Signed-off-by: Uri Simchoni 
Reviewed-by: Jeremy Allison 

Autobuild-User(master): Jeremy Allison 
Autobuild-Date(master): Wed Jan 25 04:23:00 CET 2017 on sn-devel-144

[SCM] Samba Shared Repository - branch v4-6-test updated

[Stefan Metzmacher]

The branch, v4-6-test has been updated
   via  4817385 script/release.sh: fix off by 1 error in 
announce.${tagname}.mail.txt creation
  from  d5eebe5 winbind: Don't add duplicate IDs in wbinfo -r

https://git.samba.org/?p=samba.git;a=shortlog;h=v4-6-test


- Log -
commit 4817385baccf67e962572c7f1c293dde81e14d19
Author: Stefan Metzmacher 
Date:   Thu Jan 12 10:40:37 2017 +0100

script/release.sh: fix off by 1 error in announce.${tagname}.mail.txt 
creation

Pair-Programmed-With: Karolin Seeger 

Signed-off-by: Stefan Metzmacher 
Signed-off-by: Karolin Seeger 

Autobuild-User(master): Karolin Seeger 
Autobuild-Date(master): Thu Jan 12 15:34:25 CET 2017 on sn-devel-144

(cherry picked from commit 7870c645b79da647bae45b4dc95e7d6e9abcd91a)

Autobuild-User(v4-6-test): Stefan Metzmacher 
Autobuild-Date(v4-6-test): Thu Jan 26 04:06:28 CET 2017 on sn-devel-144

---

Summary of changes:
 script/release.sh | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/script/release.sh b/script/release.sh
index 769f742..b533e25 100755
--- a/script/release.sh
+++ b/script/release.sh
@@ -830,7 +830,8 @@ announcement_samba_stable() {
top=$(cat ${TMPDIR}/WHATSNEW.txt | wc -l)
}
local skip=$(cat ${TMPDIR}/WHATSNEW.txt | grep -n '^[^ ]' | 
head -1 | cut -d ':' -f1)
-   local bottom=$(expr ${top} - \( ${skip} - 1 \))
+   local headlimit=$(expr ${top} - 1 )
+   local taillimit=$(expr ${headlimit} - \( ${skip} - 1 \))
 
cat "announce.${tagname}.quotation.txt"
echo ""
@@ -838,7 +839,7 @@ announcement_samba_stable() {
echo "Release Announcements"
echo "-"
echo ""
-   head -${top} ${TMPDIR}/WHATSNEW.txt | tail -${bottom}
+   head -${headlimit} ${TMPDIR}/WHATSNEW.txt | tail -${taillimit}
echo ""
echo ""
echo "Download Details"


-- 
Samba Shared Repository

[SCM] Samba Shared Repository - branch v4-6-test updated

[Karolin Seeger]

The branch, v4-6-test has been updated
   via  d5eebe5 winbind: Don't add duplicate IDs in wbinfo -r
   via  4773e25 winbind: Fix a typo
   via  1166de1 s3/winbindd: fix invalid free
   via  6a87647 winbind: Fix CID 1398534 Dereference before null check
   via  8c49f54 winbind: Fix CID 1398530 Resource leak
   via  bfd0fb3 winbind: Fix CID 1398530 Resource leak
   via  f629f59 winbind: Fix CID 1398531 Resource leak
   via  ff102c9 winbind: Fix CID 1398533 Resource leak
   via  bd82056 winbind: Fix CID 1398533 Resource leak
   via  1a234f5 WHATSNEW: document winbind changes
   via  56e9090 vfs_default: unlock the right file in copy chunk
   via  479fd27 ctdb-tests: Add "13.per_ip_routing shutdown" test
   via  bcdf945 ctdb-scripts: Fix regression when cleaning up routing table 
IDs
   via  5319e50 ctdb-daemon: Remove stale eventd socket
   via  e3c4968 ctdb-scripts: Fix remaining uses of "ctdb gratiousarp"
   via  77a80b7 ctdb-tests: Add takeover helper tests with 
banned/disconnected nodes
   via  e733776 ctdb-takeover: Handle case where there are no RELEASE_IPs 
to send
   via  11841d3 ctdb-takeover: Known and available IP lists should be the 
same size as nodemap
   via  c331736 ctdb-common: Add wait_send/wait_recv to sock_daemon_funcs
   via  a13e48f ctdb-common: Avoid any processing after finishing tevent_req
   via  2930832 ctdb-common: Pass tevent_req to the computation 
sub-functions
   via  6b67083 ctdb-common: Use consistent naming for sock_daemon_run 
computation functions
   via  bc79bda ctdb-common: Correct name of sock_daemon_run_send/recv 
state structure
   via  bc8e36a ctdb-tests: Add robust mutex test
   via  f1c8b35 ctdb-locking: Explicitly unlock record/db in lock helper
   via  cb31b71 ctdb-locking: Remove support for locking multiple databases
   via  28ed3cd python/schema: fix tests flapping due to oid collision
  from  63b9e1c messaging: Fix dead but not cleaned-up-yet destination 
sockets

https://git.samba.org/?p=samba.git;a=shortlog;h=v4-6-test


- Log -
commit d5eebe5e7c3c1c037218de87cb92e2bc6509fa97
Author: Volker Lendecke 
Date:   Wed Jan 18 16:54:03 2017 +0100

winbind: Don't add duplicate IDs in wbinfo -r

We look at the netsamlogon_cache entry twice: Once in queryuser and
once in lookupusergroups_cached. This can add the group SID twice.

Use add_sid_to_array_unique to avoid this.

Signed-off-by: Volker Lendecke 
Reviewed-by: Jeremy Allison 

Autobuild-User(master): Jeremy Allison 
Autobuild-Date(master): Tue Jan 24 02:36:19 CET 2017 on sn-devel-144

The last 9 patches address
BUG: https://bugzilla.samba.org/show_bug.cgi?id=12538
Backport winbind fixes.

Autobuild-User(v4-6-test): Karolin Seeger 
Autobuild-Date(v4-6-test): Thu Jan 26 00:17:35 CET 2017 on sn-devel-144

commit 4773e25a5b206895aa8b8990097fc83bed0fceb5
Author: Volker Lendecke 
Date:   Fri Jan 13 07:33:24 2017 +0100

winbind: Fix a typo

Signed-off-by: Volker Lendecke 
Reviewed-by: Jeremy Allison 

commit 1166de1cb0e9e22f7f980844c773b475aa23973b
Author: Aurelien Aptel 
Date:   Tue Jan 17 14:39:03 2017 +0100

s3/winbindd: fix invalid free

coverity fix.

TALLOC_FREE() might be called on uninitialized 'rids' at the end of the
function in case of an early error. Initialize it to NULL to turn the
TALLOC_FREE() to a noop in this case.

Signed-off-by: Aurelien Aptel 
Reviewed-by: Volker Lendecke 
Reviewed-by: David Disseldorp 

Autobuild-User(master): Volker Lendecke 
Autobuild-Date(master): Wed Jan 18 17:19:39 CET 2017 on sn-devel-144

commit 6a8764730d0b8e0f1bcadea2d2df96d712969abb
Author: Jeremy Allison 
Date:   Wed Jan 11 11:52:44 2017 -0800

winbind: Fix CID 1398534 Dereference before null check

Make all query_user_list backends consistent.

Signed-off-by: Jeremy Allison 
Reviewed-by: Andreas Schneider 

Autobuild-User(master): Andreas Schneider 
Autobuild-Date(master): Fri Jan 13 13:33:37 CET 2017 on sn-devel-144

commit 8c49f5421ff76bdd25cdba73bd8dda844895b609
Author: Volker Lendecke 
Date:   Tue Jan 10 13:29:38 2017 +

winbind: Fix CID 1398530 Resource leak

Signed-off-by: Volker Lendecke 
Reviewed-by: Jeremy Allison 

Autobuild-User(master): Jeremy Allison 
Autobuild-Date(master): Wed Jan 11 04:38:25 CET 2017 on sn-devel-144

commit bfd0fb3b28427b84263f2939d5c4ca7492b74b36
Author: 

[SCM] Samba Shared Repository - branch v4-5-test updated

[Karolin Seeger]

The branch, v4-5-test has been updated
   via  b8ae9cd vfs_default: unlock the right file in copy chunk
   via  4426d43 ctdb-tests: Add "13.per_ip_routing shutdown" test
   via  88ef026 ctdb-scripts: Fix regression when cleaning up routing table 
IDs
   via  5650e20 ctdb-scripts: Fix remaining uses of "ctdb gratiousarp"
   via  3d55b17 ctdb-tests: Add robust mutex test
   via  8fe2fc0 ctdb-locking: Explicitly unlock record/db in lock helper
   via  727f5d4 ctdb-locking: Remove support for locking multiple databases
  from  430cad1 VERSION: Bump version up to 4.5.5...

https://git.samba.org/?p=samba.git;a=shortlog;h=v4-5-test


- Log -
commit b8ae9cd9b5401c4ca973ded5453801d438e6ff64
Author: Björn Jacke 
Date:   Thu Jan 19 21:51:41 2017 +0100

vfs_default: unlock the right file in copy chunk

Signed-off-by: Bjoern Jacke 
Reviewed-by: David Disseldorp 

Autobuild-User(master): Björn Jacke 
Autobuild-Date(master): Sat Jan 21 17:00:54 CET 2017 on sn-devel-144

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12535

(cherry picked from commit 5059c8e2e3a6159bc2917ddd80d09fab35b39e66)

Autobuild-User(v4-5-test): Karolin Seeger 
Autobuild-Date(v4-5-test): Wed Jan 25 23:47:29 CET 2017 on sn-devel-144

commit 4426d4343710dded806e02b3d763614f3393dfb3
Author: Martin Schwenke 
Date:   Mon Jan 16 11:08:51 2017 +1100

ctdb-tests: Add "13.per_ip_routing shutdown" test

Ensure that it doesn't mangle the rt_tables file.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12516

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs 

Autobuild-User(master): Amitay Isaacs 
Autobuild-Date(master): Tue Jan 17 06:02:23 CET 2017 on sn-devel-144

(cherry picked from commit eaa508b82650197a7d473a24b3362e9e9c329937)

commit 88ef026ec67c06e2c0a36aaa128c7be54f0cc5f2
Author: Martin Schwenke 
Date:   Mon Jan 16 07:24:15 2017 +1100

ctdb-scripts: Fix regression when cleaning up routing table IDs

Commit 0ca00267cd2620a14968961738bcd2a69b597e95 removed explicit
continuations in strings for awk programs.  In one case this causes a
disconnect between condition and action, where an implicit
continuation does not work.  This results in duplicate lines in the
rt_tables file.

Move the opening brace for the action to make the implicit
continuation work as expected.

An alternative would be to revert the removal of the explicit
continuations and add shellcheck tags.  However, that doesn't mean
that an author of future code will necessarily use explicit
continuations, so the same mistake might still be make in the future.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12516

Reported-by: Barry Evans 
Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs 
(cherry picked from commit f9368f8e129cb32ee30cb6501a6fe728db37e1d5)

commit 5650e208de9177ac166e56ff2b7767baad8e014d
Author: Martin Schwenke 
Date:   Mon Jan 16 13:38:50 2017 +1100

ctdb-scripts: Fix remaining uses of "ctdb gratiousarp"

This changed to "ctdb gratarp" some time ago but the scripts were
never updated.

Fix the documentation for the ctdb tool too.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12512

Reported-by: Ralph Böhme 
Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs 
(cherry picked from commit 5e00a6b346325f52e35b9785eaffd72239aebcf5)

commit 3d55b17d28724cb5ddb5a6fb7b6a7b5cd28bdc65
Author: Amitay Isaacs 
Date:   Fri Dec 2 15:11:20 2016 +1100

ctdb-tests: Add robust mutex test

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12469

This demonstrates robust mutex bug on linux/glibc system.

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke 

Autobuild-User(master): Amitay Isaacs 
Autobuild-Date(master): Thu Jan 12 07:59:34 CET 2017 on sn-devel-144

(cherry picked from commit 7794497bc909fa7b02da9d9ce1fc496a8fa2a9ae)

commit 8fe2fc0b9e6be7166062f6802e0af1740fbb2f5a
Author: Amitay Isaacs 
Date:   Tue Nov 29 17:20:45 2016 +1100

ctdb-locking: Explicitly unlock record/db in lock helper

BUG: https://bugzilla.samba.org/show_bug.cgi?id=12469

Instead of killing lock helper processes with SIGKILL, send SIGTERM so
lock helper processes can explicitly unlock record/db.

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin