summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* VERSION: Disable git snapshots for the 4.5.0rc2 release.samba-4.5.0rc2Karolin Seeger2016-08-101-1/+1
| | | | Signed-off-by: Karolin Seeger <kseeger@samba.org>
* WHATSNEW: Add changes since rc1.Karolin Seeger2016-08-101-3/+33
| | | | Signed-off-by: Karolin Seeger <kseeger@samba.org>
* dbwrap_ctdb: treat empty records in ltdb as non-existingRalph Boehme2016-08-101-1/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When fetching records from remote ctdb nodes via ctdbd_parse() or in db_ctdb_traverse(), we already check for tombstone records and skip them. This was originally also done for the ltdb checks. See also bug: https://bugzilla.samba.org/show_bug.cgi?id=10008 (commit 1cae59ce112ccb51b45357a52b902f80fce1eef1). Commit 925625b52886d40b50fc631bad8bdc81970f7598 reverted part of the patch of bug 10008 due to a deadlock it introduced. This patch re-introduces the consistent treatment of empty records in the ltdb but avoids the deadlock by correctly signalling NT_STATUS_NOT_FOUND if an empty record is found authoritatively in the ltdb and not calling ctdb in this case. BUG: https://bugzilla.samba.org/show_bug.cgi?id=12005 Pair-Programmed-With: Michael Adam <obnox@samba.org> Signed-off-by: Ralph Boehme <slow@samba.org> Signed-off-by: Michael Adam <obnox@samba.org> Autobuild-User(master): Michael Adam <obnox@samba.org> Autobuild-Date(master): Tue Aug 9 04:38:44 CEST 2016 on sn-devel-144 (cherry picked from commit 25df582739918b7afd4e5497eaffe279e2d92cd1) Autobuild-User(v4-5-test): Stefan Metzmacher <metze@samba.org> Autobuild-Date(v4-5-test): Wed Aug 10 15:08:00 CEST 2016 on sn-devel-144
* s4/torture: add a test for ctdb-tombstrone-record deadlockRalph Boehme2016-08-101-0/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This tests for a possible deadlock between smbd and ctdb dealing with ctdb tombstone records. Commit 925625b52886d40b50fc631bad8bdc81970f7598 explains the deadlock in more details and contains the fix. It's a fix for a regression introduced by the patch for bug 10008 (1cae59ce112c). If you ever want to use this test against that specific commit: $ git checkout 925625b52886d40b50fc631bad8bdc81970f7598 $ git cherry-pick THIS_COMMIT This should not deadlock on a ctdb cluster. $ git revert 925625b52886d40b50fc631bad8bdc81970f7598 This will deadlock. Bug: https://bugzilla.samba.org/show_bug.cgi?id=12005 Pair-Programmed-With: Michael Adam <obnox@samba.org> Signed-off-by: Ralph Boehme <slow@samba.org> Signed-off-by: Michael Adam <obnox@samba.org> (cherry picked from commit b17e2f5c740fb081c007ed2e1c23138ffcba1469)
* smbd: ignore ctdb tombstone records in fetch_share_mode_unlocked_parser()Ralph Boehme2016-08-101-0/+6
| | | | | | | | | | | dbwrap_parse_record() can return ctdb tombstone records from the lctdb, ignore them. Bug: https://bugzilla.samba.org/show_bug.cgi?id=12005 Signed-off-by: Ralph Boehme <slow@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org> (cherry picked from commit 7147859c7afc1344e76485e2cbc286679110d96e)
* dbwrap: Fix structure initializationAmitay Isaacs2016-08-101-1/+2
| | | | | | | | | | | Bug: https://bugzilla.samba.org/show_bug.cgi?id=12134 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Volker Lendecke <vl@samba.org> Autobuild-User(master): Volker Lendecke <vl@samba.org> Autobuild-Date(master): Tue Aug 9 15:15:43 CEST 2016 on sn-devel-144 (cherry picked from commit afa41f023b4c3af067e401e53d2e89a8d568814a)
* buildscripts: Fix the regression with --without-acl-support.Ira Cooper2016-08-102-4/+12
| | | | | | | | | | | | | | | | | | | This will disable the vfs_glusterfs and vfs_cephfs modules if you provide --without-acl-support. In addition it makes compiling in POSIXACL_XATTR conditional on vfs_glusterfs or vfs_cephfs being built in. Signed-off-by: Ira Cooper <ira@samba.org> Reviewed-by: Steve French <sfrench@samba.org> Tested-by: Steve French <sfrench@samba.org> Autobuild-User(master): Ira Cooper <ira@samba.org> Autobuild-Date(master): Mon Aug 8 21:27:46 CEST 2016 on sn-devel-144 BUG: https://bugzilla.samba.org/show_bug.cgi?id=12133 (cherry picked from commit 14f29c4f560f1b3942245a6fa43924a8ce45d99d)
* source3/wscript: Add support for disabling vfs_cephfsIra Cooper2016-08-101-0/+1
| | | | | | | | | | | | | | | --enable-cephfs and --disable-cephfs both work now. --enable-cephfs is the default to match previous versions of Samba. Signed-off-by: Ira Cooper <ira@samba.org> Reviewed-by: Steve French <sfrench@samba.org> Tested-by: Steve French <sfrench@samba.org> BUG: https://bugzilla.samba.org/show_bug.cgi?id=12133 (cherry picked from commit 38bb993657f8eb6c49899de0d741ed5186ca5022)
* ctdb-daemon: Clean up SET_DB_PRIORITY/GET_DB_PRIORITY deprecationMartin Schwenke2016-08-101-2/+2
| | | | | | | | | | | | The current message is broken: Control SET_DB_PRIORITY is not implemented any more, use instead BUG: https://bugzilla.samba.org/show_bug.cgi?id=12126 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit d7ecc913bb40783c8cf8c66e3c3e99e6df414381)
* ctdb-packaging: Stop RPM from renaming working config to ctdb.rpmsaveMartin Schwenke2016-08-101-0/+11
| | | | | | | | | | | | | | | | | | | | The change to ctdbd.conf and removal of /etc/sysconfig/ctdb as a configuration file makes RPM rename the latter with a .rpmsave suffix. This means that a working configuration is moved aside on upgrade and manual intervention is needed. Avoid this by convincing RPM that the existing /etc/sysconfig/ctdb is still a configuration file. BUG: https://bugzilla.samba.org/show_bug.cgi?id=12125 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> Autobuild-User(master): Martin Schwenke <martins@samba.org> Autobuild-Date(master): Mon Aug 8 11:30:58 CEST 2016 on sn-devel-144 (cherry picked from commit 464b386d55b0c509b86645bdac7f6d3df489e902)
* ctdb-doc: Document that "ctdb tickle" can now read from stdinMartin Schwenke2016-08-101-6/+17
| | | | | | | | | | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12123 Although this has been queued for a while, with changes to add stdin support to the original ctdb tool, this came for free with the replacement tool. addtickle and deltickle also now have this feature. However, they're internal commands and we're not adding new documentation for internal/debug commands. Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 2295f5009dabfc09eec40bb4f30679d76310ce12)
* ctdb-tools: "ctdb tickle" command should run without daemonMartin Schwenke2016-08-101-1/+1
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12123 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit f2b319e457eb43410997615f834f0a21fa5e5b0a)
* ctdb-tools: Cancel transaction on error or if commit failsAmitay Isaacs2016-08-101-0/+6
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12122 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 4d144d7106d56bbb48669dd35eb324377dd09b59)
* ctdb-client: transaction_cancel must free transaction handleAmitay Isaacs2016-08-101-0/+3
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12122 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 8aa00035faeb307c061dc20e62a0ace7f2a1413b)
* ctdb-tools: Free temporary memory context before exitingAmitay Isaacs2016-08-101-1/+3
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 3c51ac48df50a877ded34f3678aeb13f3509570d)
* ctdb-tools: Close tdb database on errorAmitay Isaacs2016-08-101-0/+6
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit c9750c821e06364cb4559f857c359c14e6799303)
* ctdb-tools: Free connection list after processing itAmitay Isaacs2016-08-101-0/+4
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 2e5e51ebcd3ec8283cdd9c9d13dbe1fc836f8a3d)
* ctdb-tools: Add early return for empty connection listMartin Schwenke2016-08-101-0/+6
| | | | | | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 If the connection list is empty then process_clist_send() still creates a request. However, since no subrequests are created for controls sent, tevent_req_poll() waits forever for an event. Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 1f6835c73245366111946b92605315c0cac84295)
* ctdb-tools: Free tickle list before exitingAmitay Isaacs2016-08-101-0/+1
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit b54bd763b7a4ff3cd1983a8ccc89b23db5263a4d)
* ctdb-tools: Free record if it does not contain valid dataAmitay Isaacs2016-08-101-0/+1
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 814002c9a10227fd6cc13fdc974d2312201d7c19)
* ctdb-tools: Fix CID 1364706 - resource leakAmitay Isaacs2016-08-101-0/+4
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit bfe891e418fd372c1e7036f08e81b65c27843a79)
* ctdb-tools: Fix CID 1364705 - resource leakAmitay Isaacs2016-08-101-0/+2
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 5f5d9549fcae4b2fa4b99bef3892a24bdbb0eaa9)
* ctdb-tools: Fix CID 1364704 - resource leakAmitay Isaacs2016-08-101-0/+1
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 45b9c379b590ce0ff03ccaf6c970e0aee1a4b745)
* ctdb-tools: Fix CID 1364703 - resource leakAmitay Isaacs2016-08-101-0/+4
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 94bf6e69dba1b97d90168efceaafa43d27186778)
* ctdb-tools: Fix CID 1364702 - resource leakAmitay Isaacs2016-08-101-0/+2
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 10019f7cdfa4b3f5485f063d409aa55ca5f14f6a)
* ctdb-tools: Fix CID 1364701 - resource leakAmitay Isaacs2016-08-101-6/+5
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 25f27e16b26e2c96a35dd7b42d22d16d63be6037)
* ctdb-tools: Fix CID 1364699 - dereference after null checkAmitay Isaacs2016-08-101-0/+2
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12121 Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit bf1dab28d1d5887f39739eb29537727d97054c5d)
* ctdb-mutex: Avoid corner case where helper is already reparented to initMartin Schwenke2016-08-102-1/+15
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12113 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 1f942ec36c98127e25d7aa9b8370a4f621b9d362)
* ctdb-daemon: Fix CID 1125627 Resource leak (RESOURCE_LEAK)Martin Schwenke2016-08-101-2/+7
| | | | | | | | | | Also fixes CID 1125628. BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 940272d215049f5f5079aa926e69eae1985a4bfa)
* ctdb-common: Fix CID 1125585 Dereference after null check (FORWARD_NULL)Martin Schwenke2016-08-101-1/+1
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit b4f23a7e95cd9c8fc4a6324d4ec5a2881eaec207)
* ctdb-common: Fix CID 1125583 Dereference after null check (FORWARD_NULL)Martin Schwenke2016-08-101-1/+1
| | | | | | | | | | This also fixes CID 1125584. BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit bbf0b907cb04184515d0f5f09f14824df1c2e59f)
* ctdb-common: Fix CID 1125581 Dereference after null check (FORWARD_NULL)Martin Schwenke2016-08-101-1/+1
| | | | | | | | | | This also fixes CID 1125582. BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 7ae3699831427725f12e0a26a0681e59f2fbb2d9)
* ctdb-tests: Fix CID 1361816 Buffer not null terminated (BUFFER_SIZE_WARNING)Martin Schwenke2016-08-101-2/+2
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit f802a11d3ba808baba6cb2d31d706952df1e3159)
* ctdb-common: Fix CID 1363227 Resource leak (RESOURCE_LEAK)Martin Schwenke2016-08-101-0/+1
| | | | | | | | | | | | Occurs on an invalid line that isn't the first. Isn't really a leak because it is allocated off mem_ctx, which should be freed by the caller. BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 4541301d92e3c140adb6444143ad697d35926179)
* ctdb-mutex: Fix CID 1359217 Resource leak (RESOURCE_LEAK)Martin Schwenke2016-08-101-2/+9
| | | | | | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> Autobuild-User(master): Amitay Isaacs <amitay@samba.org> Autobuild-Date(master): Wed Aug 3 09:13:55 CEST 2016 on sn-devel-144 (cherry picked from commit 24e28c0aa52399f03acf830a1ad72af0139b0606)
* ctdb-daemon: Fix CID 1363067 Resource leak (RESOURCE_LEAK)Martin Schwenke2016-08-101-0/+1
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit c6a7f680ce74d4a630fa9305d0a926cc1a4b3d2c)
* ctdb-daemon: Fix CID 1363233 Resource leak (RESOURCE_LEAK)Martin Schwenke2016-08-101-0/+1
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 74aca5f4c671d9f15ae6c3a901978a1cf247dd6f)
* ctdb-utils: Fix CID 1297451 Explicit null dereferenced (FORWARD_NULL)Martin Schwenke2016-08-101-1/+3
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit d46960f20e795cd4abc1c727705f77b2f0e0e564)
* ctdb-common: Consistently use strlcpy() on interface namesMartin Schwenke2016-08-101-3/+3
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit aff33a59479cafcb1f24a07ff76383d47bb196b3)
* ctdb-common: Fix CID 1125553 Buffer not null terminated (BUFFER_SIZE_WARNING)Martin Schwenke2016-08-101-1/+1
| | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12110 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 763f9c13f2998a8858e8a3ec013d166a3d429835)
* ctdb-tests: Add explicit wait to the fork_helper()Amitay Isaacs2016-08-101-11/+6
| | | | | | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 To create a client process that waits after connect(), instead of trying to do a blocking write(), go to sleep. The parent can then kill the client process, once testing is done. Signed-off-by: Amitay Isaacs <amitay@gmail.com> Reviewed-by: Martin Schwenke <martin@meltin.net> (cherry picked from commit 049dd07b909dc56a246799f8a96235989a5a605f)
* ctdb-tests: Clean up and rename simple transaction_loop recovery testMartin Schwenke2016-08-101-14/+4
| | | | | | | | | | | | | | | * Rename to clarify purpose of test * Simplify test info to avoid unnecessary bit-rot * Restart after test since test does heavy database manipulation BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 8ce8e1d7f8c05158f0f5f4e673f3d9a40f94f00d)
* ctdb-tests: Clean up and rename simple transaction_loop testMartin Schwenke2016-08-101-13/+2
| | | | | | | | | | | | | * Rename to clarify purpose of test * Simplify test info to avoid unnecessary bit-rot BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit de04cd578416168290c51f7c4e326b864ab425a4)
* ctdb-tests: Implement --interactive/-i option in transaction_loopMartin Schwenke2016-08-101-6/+13
| | | | | | | | | | | Also add PNNs to output. BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 18b41fcb51942ebcd4041fe3e3694712a5a5e43a)
* ctdb-tests: Clean up and rename simple fetch_ring testMartin Schwenke2016-08-103-65/+48
| | | | | | | | | | | | | | | | | | | * Rename to clarify purpose of test * Simplify test info to avoid unnecessary bit-rot * Have fetch_ring print PNN for clearer output and update patterns in test script to suit * Simplify sanity checking pattern due to less data because of --interactive/-i option BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit c10dcc7b91df62f740a4789763234f8bb2f0a27f)
* ctdb-tests: Implement --interactive/-i option in fetch ringMartin Schwenke2016-08-101-3/+9
| | | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit 82cffc2f5759fb73bba37e45fc20b24b30ab0438)
* ctdb-tests: Clean up and rename simple message_ring testMartin Schwenke2016-08-103-94/+69
| | | | | | | | | | | | | | | | | | | | | * Rename to clarify purpose of test * Simplify test info to avoid unnecessary bit-rot * Have message_ring print PNN for clearer output and update patterns in test script to suit. * Drop quantitative percentage check since this is hard to predict when under extreme load. To compensate, tighten up expectations for the total number of messages in each direction: at least 10 messages are required over 10 seconds. BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit e7a220377ec14090435f448c9a900f8f8e852a77)
* ctdb-tests: Implement --interactive/-i option in message_ringMartin Schwenke2016-08-101-5/+9
| | | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit fbb3ef46b99f4e3ccf710367e50f8b9f89fee52c)
* ctdb-tests: Add --interactive/-i option to test options parsing codeMartin Schwenke2016-08-102-0/+4
| | | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 Pair-programmed-with: Amitay Isaacs <amitay@gmail.com> Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Amitay Isaacs <amitay@gmail.com> (cherry picked from commit b100d78597959ad314e4f4f6816a8ff14100fb6e)
* ctdb-tests: Stop cross-talk between reclock testsMartin Schwenke2016-08-101-0/+11
| | | | | | | | | | | | | | | | | | On a busy system the backgrounded counter reset can survive into the next test and interfere with its result. To avoid this, wait until all forks of 01.reclock exit before continuing on to the next test. BUG: https://bugzilla.samba.org/show_bug.cgi?id=12109 Signed-off-by: Martin Schwenke <martin@meltin.net> Reviewed-by: Volker Lendecke <vl@samba.org> Autobuild-User(master): Volker Lendecke <vl@samba.org> Autobuild-Date(master): Mon Aug 1 17:53:27 CEST 2016 on sn-devel-144 (cherry picked from commit 19ed8165bbe374e1ea277fd4dd5a65ee932b4f05)