diff options
author | Martin Schwenke <martin@meltin.net> | 2019-07-29 16:45:07 +1000 |
---|---|---|
committer | Martin Schwenke <martins@samba.org> | 2019-08-21 11:50:30 +0000 |
commit | 53daeb2f878af1634a26e05cb86d87e2faf20173 (patch) | |
tree | aa67fccfe4f8565aeaa0c39ca8047a234eac4278 /ctdb | |
parent | bff1a3a548a2cace997b767d78bb824438664cb7 (diff) | |
download | samba-53daeb2f878af1634a26e05cb86d87e2faf20173.tar.gz |
ctdb-tests: Don't retrieve the VNN map from target node for notlmaster
Use the VNN map from the node running node_has_status().
This means that
wait_until_node_has_status 1 notlmaster 10 0
will run "ctdb status" on node 0 and check (for up to 10 seconds) if
node 1 is in the VNN map.
If the LMASTER capability has been dropped on node 1 then the above
will wait for the VNN map to be updated on node 0. This will happen
as part of the recovery that is triggered by the change of LMASTER
capability. The next command will then only be able to attach to
$TESTDB after the recovery is complete thus guaranteeing a sane state
for the test to continue.
This stops simple/79_volatile_db_traverse.sh from going into recovery
during the traverse or at some other inconvenient time.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=14085
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Diffstat (limited to 'ctdb')
-rw-r--r-- | ctdb/tests/scripts/integration.bash | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ctdb/tests/scripts/integration.bash b/ctdb/tests/scripts/integration.bash index 36ee09c563b..284449d4503 100644 --- a/ctdb/tests/scripts/integration.bash +++ b/ctdb/tests/scripts/integration.bash @@ -329,8 +329,7 @@ node_has_status () return ;; notlmaster) - ! $CTDB status -n "$pnn" | \ - grep -Eq "^hash:.* lmaster:${pnn}\$" + ! $CTDB status | grep -Eq "^hash:.* lmaster:${pnn}\$" return ;; esac |