[Tarantool-patches] [PATCH v2 4/4] test: stabilize quorum test conditions
Ilya Kosarev
i.kosarev at tarantool.org
Sun Nov 24 00:53:58 MSK 2019
There were some pass conditions in quorum test which could take some
time to be satisfied. Now they are wrapped using test_run:wait_cond to
make the test stable.
Closes #4586
---
test/replication/quorum.result | 54 ++++++++++++++++++++++++--------
test/replication/quorum.test.lua | 24 ++++++++------
2 files changed, 56 insertions(+), 22 deletions(-)
diff --git a/test/replication/quorum.result b/test/replication/quorum.result
index ff5fa0150..939ce1e00 100644
--- a/test/replication/quorum.result
+++ b/test/replication/quorum.result
@@ -40,6 +40,10 @@ box.info.ro -- true
---
- true
...
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
+---
+- true
+...
box.space.test:replace{100} -- error
---
- error: Can't modify data because this instance is in read-only mode.
@@ -64,6 +68,10 @@ box.info.ro -- true
---
- true
...
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
+---
+- true
+...
box.space.test:replace{100} -- error
---
- error: Can't modify data because this instance is in read-only mode.
@@ -95,6 +103,10 @@ box.info.ro -- true
---
- true
...
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
+---
+- true
+...
box.space.test:replace{100} -- error
---
- error: Can't modify data because this instance is in read-only mode.
@@ -115,15 +127,15 @@ box.info.status -- running
- running
...
-- Check that the replica follows all masters.
-box.info.id == 1 or box.info.replication[1].upstream.status == 'follow'
+box.info.id == 1 or test_run:wait_cond(function() return box.info.replication[1].upstream.status == 'follow' end, 20)
---
- true
...
-box.info.id == 2 or box.info.replication[2].upstream.status == 'follow'
+box.info.id == 2 or test_run:wait_cond(function() return box.info.replication[2].upstream.status == 'follow' end, 20)
---
- true
...
-box.info.id == 3 or box.info.replication[3].upstream.status == 'follow'
+box.info.id == 3 or test_run:wait_cond(function() return box.info.replication[3].upstream.status == 'follow' end, 20)
---
- true
...
@@ -149,6 +161,14 @@ test_run:cmd('stop server quorum1')
---
- true
...
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
+---
+- true
+...
+test_run:wait_cond(function() return box.space.test.index.primary ~= nil end, 20)
+---
+- true
+...
for i = 1, 100 do box.space.test:insert{i} end
---
...
@@ -166,9 +186,13 @@ test_run:cmd('switch quorum1')
---
- true
...
-box.space.test:count() -- 100
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
---
-- 100
+- true
+...
+test_run:wait_cond(function() return box.space.test:count() == 100 end, 20)
+---
+- true
...
-- Rebootstrap one node of the cluster and check that others follow.
-- Note, due to ERRINJ_RELAY_TIMEOUT there is a substantial delay
@@ -197,9 +221,13 @@ test_run:cmd('switch quorum1')
- true
...
test_run:cmd('restart server quorum1 with cleanup=1, args="0.1 0.5"')
-box.space.test:count() -- 100
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
---
-- 100
+- true
+...
+test_run:wait_cond(function() return box.space.test:count() == 100 end, 20)
+---
+- true
...
-- The rebootstrapped replica will be assigned id = 4,
-- because ids 1..3 are busy.
@@ -207,11 +235,9 @@ test_run:cmd('switch quorum2')
---
- true
...
-fiber = require('fiber')
----
-...
-while box.info.replication[4].upstream.status ~= 'follow' do fiber.sleep(0.001) end
+test_run:wait_cond(function() return box.info.replication[4].upstream.status == 'follow' end, 20)
---
+- true
...
box.info.replication[4].upstream.status
---
@@ -221,11 +247,13 @@ test_run:cmd('switch quorum3')
---
- true
...
-fiber = require('fiber')
+test_run:wait_cond(function() return box.info.replication ~= nil end, 20)
---
+- true
...
-while box.info.replication[4].upstream.status ~= 'follow' do fiber.sleep(0.001) end
+test_run:wait_cond(function() return box.info.replication[4].upstream.status == 'follow' end, 20)
---
+- true
...
box.info.replication[4].upstream.status
---
diff --git a/test/replication/quorum.test.lua b/test/replication/quorum.test.lua
index 98febb367..e20572344 100644
--- a/test/replication/quorum.test.lua
+++ b/test/replication/quorum.test.lua
@@ -22,6 +22,7 @@ test_run:cmd('restart server quorum2 with args="0.1 0.5"')
box.info.status -- orphan
box.ctl.wait_rw(0.001) -- timeout
box.info.ro -- true
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
box.space.test:replace{100} -- error
box.cfg{replication={}}
box.info.status -- running
@@ -30,6 +31,7 @@ test_run:cmd('restart server quorum2 with args="0.1 0.5"')
box.info.status -- orphan
box.ctl.wait_rw(0.001) -- timeout
box.info.ro -- true
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
box.space.test:replace{100} -- error
box.cfg{replication_connect_quorum = 2}
box.ctl.wait_rw()
@@ -40,6 +42,7 @@ test_run:cmd('restart server quorum2 with args="0.1 0.5"')
box.info.status -- orphan
box.ctl.wait_rw(0.001) -- timeout
box.info.ro -- true
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
box.space.test:replace{100} -- error
test_run:cmd('start server quorum1 with args="0.1 0.5"')
box.ctl.wait_rw()
@@ -47,9 +50,9 @@ box.info.ro -- false
box.info.status -- running
-- Check that the replica follows all masters.
-box.info.id == 1 or box.info.replication[1].upstream.status == 'follow'
-box.info.id == 2 or box.info.replication[2].upstream.status == 'follow'
-box.info.id == 3 or box.info.replication[3].upstream.status == 'follow'
+box.info.id == 1 or test_run:wait_cond(function() return box.info.replication[1].upstream.status == 'follow' end, 20)
+box.info.id == 2 or test_run:wait_cond(function() return box.info.replication[2].upstream.status == 'follow' end, 20)
+box.info.id == 3 or test_run:wait_cond(function() return box.info.replication[3].upstream.status == 'follow' end, 20)
-- Check that box.cfg() doesn't return until the instance
-- catches up with all configured replicas.
@@ -59,13 +62,16 @@ test_run:cmd('switch quorum2')
box.error.injection.set("ERRINJ_RELAY_TIMEOUT", 0.001)
test_run:cmd('stop server quorum1')
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
+test_run:wait_cond(function() return box.space.test.index.primary ~= nil end, 20)
for i = 1, 100 do box.space.test:insert{i} end
fiber = require('fiber')
fiber.sleep(0.1)
test_run:cmd('start server quorum1 with args="0.1 0.5"')
test_run:cmd('switch quorum1')
-box.space.test:count() -- 100
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
+test_run:wait_cond(function() return box.space.test:count() == 100 end, 20)
-- Rebootstrap one node of the cluster and check that others follow.
-- Note, due to ERRINJ_RELAY_TIMEOUT there is a substantial delay
@@ -81,17 +87,17 @@ box.snapshot()
test_run:cmd('switch quorum1')
test_run:cmd('restart server quorum1 with cleanup=1, args="0.1 0.5"')
-box.space.test:count() -- 100
+test_run:wait_cond(function() return box.space.test ~= nil end, 20)
+test_run:wait_cond(function() return box.space.test:count() == 100 end, 20)
-- The rebootstrapped replica will be assigned id = 4,
-- because ids 1..3 are busy.
test_run:cmd('switch quorum2')
-fiber = require('fiber')
-while box.info.replication[4].upstream.status ~= 'follow' do fiber.sleep(0.001) end
+test_run:wait_cond(function() return box.info.replication[4].upstream.status == 'follow' end, 20)
box.info.replication[4].upstream.status
test_run:cmd('switch quorum3')
-fiber = require('fiber')
-while box.info.replication[4].upstream.status ~= 'follow' do fiber.sleep(0.001) end
+test_run:wait_cond(function() return box.info.replication ~= nil end, 20)
+test_run:wait_cond(function() return box.info.replication[4].upstream.status == 'follow' end, 20)
box.info.replication[4].upstream.status
-- Cleanup.
--
2.17.1
More information about the Tarantool-patches
mailing list