From: Cyrill Gorcunov <gorcunov@gmail.com> To: tml <tarantool-patches@dev.tarantool.org> Cc: Vladislav Shpilevoy <v.shpilevoy@tarantool.org> Subject: [Tarantool-patches] [PATCH v6 5/5] test: add replication/gh-5446-qsync-eval-quorum.test.lua Date: Tue, 22 Dec 2020 14:14:08 +0300 [thread overview] Message-ID: <20201222111408.48368-6-gorcunov@gmail.com> (raw) In-Reply-To: <20201222111408.48368-1-gorcunov@gmail.com> Part-of #5446 Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com> --- .../gh-5446-qsync-eval-quorum.result | 277 ++++++++++++++++++ .../gh-5446-qsync-eval-quorum.test.lua | 110 +++++++ test/replication/replica-quorum-1.lua | 1 + test/replication/replica-quorum-2.lua | 1 + test/replication/replica-quorum-3.lua | 1 + test/replication/replica-quorum-4.lua | 1 + test/replication/replica-quorum-5.lua | 1 + test/replication/replica-quorum-6.lua | 1 + 8 files changed, 393 insertions(+) create mode 100644 test/replication/gh-5446-qsync-eval-quorum.result create mode 100644 test/replication/gh-5446-qsync-eval-quorum.test.lua create mode 120000 test/replication/replica-quorum-1.lua create mode 120000 test/replication/replica-quorum-2.lua create mode 120000 test/replication/replica-quorum-3.lua create mode 120000 test/replication/replica-quorum-4.lua create mode 120000 test/replication/replica-quorum-5.lua create mode 120000 test/replication/replica-quorum-6.lua diff --git a/test/replication/gh-5446-qsync-eval-quorum.result b/test/replication/gh-5446-qsync-eval-quorum.result new file mode 100644 index 000000000..6c3c63577 --- /dev/null +++ b/test/replication/gh-5446-qsync-eval-quorum.result @@ -0,0 +1,277 @@ +-- test-run result file version 2 +test_run = require('test_run').new() + | --- + | ... +engine = test_run:get_cfg('engine') + | --- + | ... + +box.schema.user.grant('guest', 'replication') + | --- + | ... + +-- Test syntax error +box.cfg{replication_synchro_quorum = "aaa"} + | --- + | - error: 'Incorrect value for option ''replication_synchro_quorum'': [string "local + | expr = [[aaa]]..."]:7: Expression should return a number' + | ... + +-- Test out of bounds values +box.cfg{replication_synchro_quorum = "N+1"} + | --- + | - error: 'Incorrect value for option ''replication_synchro_quorum'': the formula is + | evaluated to the quorum 32 for replica number 31, which is out of range [1;31]' + | ... +box.cfg{replication_synchro_quorum = "N-1"} + | --- + | - error: 'Incorrect value for option ''replication_synchro_quorum'': the formula is + | evaluated to the quorum 0 for replica number 1, which is out of range [1;31]' + | ... + +-- Test big number value +box.cfg{replication_synchro_quorum = '4294967297'} + | --- + | - error: 'Incorrect value for option ''replication_synchro_quorum'': the value must + | be greater than zero and less than maximal number of replicas' + | ... +box.cfg{replication_synchro_quorum = 4294967297} + | --- + | - error: 'Incorrect value for option ''replication_synchro_quorum'': the value must + | be greater than zero and less than maximal number of replicas' + | ... + +-- Use canonical majority formula +box.cfg { replication_synchro_quorum = "N/2+1", replication_synchro_timeout = 1000 } + | --- + | ... +match = 'set \'replication_synchro_quorum\' configuration option to \"N\\/2%+1' + | --- + | ... +test_run:wait_log('default', match, nil, 5) + | --- + | - set 'replication_synchro_quorum' configuration option to "N\/2+1 + | ... + +-- Create a sync space we will operate on +_ = box.schema.space.create('sync', {is_sync = true, engine = engine}) + | --- + | ... +_ = box.space.sync:create_index('pk') + | --- + | ... +s = box.space.sync + | --- + | ... +s:insert{1} + | --- + | - [1] + | ... + +test_run:cmd('create server replica1 with rpl_master=default,\ + script="replication/replica-quorum-1.lua"') + | --- + | - true + | ... +test_run:cmd('start server replica1 with wait=True, wait_load=True') + | --- + | - true + | ... + +-- 1 replica -> replication_synchro_quorum = 2/2 + 1 = 2 +match = 'update replication_synchro_quorum = 2' + | --- + | ... +test_run:wait_log('default', match, nil, 5) + | --- + | - update replication_synchro_quorum = 2 + | ... + +test_run:cmd('create server replica2 with rpl_master=default,\ + script="replication/replica-quorum-2.lua"') + | --- + | - true + | ... +test_run:cmd('start server replica2 with wait=True, wait_load=True') + | --- + | - true + | ... + +-- 2 replicas -> replication_synchro_quorum = 3/2 + 1 = 2 +match = 'update replication_synchro_quorum = 2' + | --- + | ... +test_run:wait_log('default', match, nil, 5) + | --- + | - update replication_synchro_quorum = 2 + | ... + +test_run:cmd('create server replica3 with rpl_master=default,\ + script="replication/replica-quorum-3.lua"') + | --- + | - true + | ... +test_run:cmd('start server replica3 with wait=True, wait_load=True') + | --- + | - true + | ... + +-- 3 replicas -> replication_synchro_quorum = 4/2 + 1 = 3 +match = 'update replication_synchro_quorum = 3' + | --- + | ... +test_run:wait_log('default', match, nil, 5) + | --- + | - update replication_synchro_quorum = 3 + | ... + +test_run:cmd('create server replica4 with rpl_master=default,\ + script="replication/replica-quorum-4.lua"') + | --- + | - true + | ... +test_run:cmd('start server replica4 with wait=True, wait_load=True') + | --- + | - true + | ... + +-- 4 replicas -> replication_synchro_quorum = 5/2 + 1 = 3 +match = 'update replication_synchro_quorum = 3' + | --- + | ... +test_run:wait_log('default', match, nil, 5) + | --- + | - update replication_synchro_quorum = 3 + | ... + +test_run:cmd('create server replica5 with rpl_master=default,\ + script="replication/replica-quorum-5.lua"') + | --- + | - true + | ... +test_run:cmd('start server replica5 with wait=True, wait_load=True') + | --- + | - true + | ... + +test_run:cmd('create server replica6 with rpl_master=default,\ + script="replication/replica-quorum-6.lua"') + | --- + | - true + | ... +test_run:cmd('start server replica6 with wait=True, wait_load=True') + | --- + | - true + | ... + +-- 6 replicas -> replication_synchro_quorum = 7/2 + 1 = 4 +match = 'update replication_synchro_quorum = 4' + | --- + | ... +test_run:wait_log('default', match, nil, 5) + | --- + | - update replication_synchro_quorum = 4 + | ... + +-- 5 replicas left, the commit should pass +test_run:cmd('stop server replica1') + | --- + | - true + | ... +test_run:cmd('delete server replica1') + | --- + | - true + | ... +s:insert{2} + | --- + | - [2] + | ... + +-- 4 replicas left,the commit should pass +test_run:cmd('stop server replica2') + | --- + | - true + | ... +test_run:cmd('delete server replica2') + | --- + | - true + | ... +s:insert{3} + | --- + | - [3] + | ... + +-- 3 replicas left, the commit should pass +test_run:cmd('stop server replica3') + | --- + | - true + | ... +test_run:cmd('delete server replica3') + | --- + | - true + | ... +s:insert{4} + | --- + | - [4] + | ... + +-- 2 replicas left, the commit should NOT pass +-- +-- The replication_synchro_timeout set to a small value to not wait +-- for very long for the case where we know the commit should +-- not pass since replicas are stopped. +box.cfg { replication_synchro_timeout = 0.5 } + | --- + | ... +test_run:cmd('stop server replica4') + | --- + | - true + | ... +s:insert{5} + | --- + | - error: Quorum collection for a synchronous transaction is timed out + | ... +-- restore it back and retry +test_run:cmd('start server replica4 with wait=True, wait_load=True') + | --- + | - true + | ... +box.cfg { replication_synchro_timeout = 1000 } + | --- + | ... +s:insert{5} + | --- + | - [5] + | ... +test_run:cmd('stop server replica4') + | --- + | - true + | ... +test_run:cmd('delete server replica4') + | --- + | - true + | ... + +-- cleanup leftovers + +test_run:cmd('stop server replica5') + | --- + | - true + | ... +test_run:cmd('delete server replica5') + | --- + | - true + | ... + +test_run:cmd('stop server replica6') + | --- + | - true + | ... +test_run:cmd('delete server replica6') + | --- + | - true + | ... + +box.schema.user.revoke('guest', 'replication') + | --- + | ... diff --git a/test/replication/gh-5446-qsync-eval-quorum.test.lua b/test/replication/gh-5446-qsync-eval-quorum.test.lua new file mode 100644 index 000000000..b7ce9a9ca --- /dev/null +++ b/test/replication/gh-5446-qsync-eval-quorum.test.lua @@ -0,0 +1,110 @@ +test_run = require('test_run').new() +engine = test_run:get_cfg('engine') + +box.schema.user.grant('guest', 'replication') + +-- Test syntax error +box.cfg{replication_synchro_quorum = "aaa"} + +-- Test out of bounds values +box.cfg{replication_synchro_quorum = "N+1"} +box.cfg{replication_synchro_quorum = "N-1"} + +-- Test big number value +box.cfg{replication_synchro_quorum = '4294967297'} +box.cfg{replication_synchro_quorum = 4294967297} + +-- Use canonical majority formula +box.cfg { replication_synchro_quorum = "N/2+1", replication_synchro_timeout = 1000 } +match = 'set \'replication_synchro_quorum\' configuration option to \"N\\/2%+1' +test_run:wait_log('default', match, nil, 5) + +-- Create a sync space we will operate on +_ = box.schema.space.create('sync', {is_sync = true, engine = engine}) +_ = box.space.sync:create_index('pk') +s = box.space.sync +s:insert{1} + +test_run:cmd('create server replica1 with rpl_master=default,\ + script="replication/replica-quorum-1.lua"') +test_run:cmd('start server replica1 with wait=True, wait_load=True') + +-- 1 replica -> replication_synchro_quorum = 2/2 + 1 = 2 +match = 'update replication_synchro_quorum = 2' +test_run:wait_log('default', match, nil, 5) + +test_run:cmd('create server replica2 with rpl_master=default,\ + script="replication/replica-quorum-2.lua"') +test_run:cmd('start server replica2 with wait=True, wait_load=True') + +-- 2 replicas -> replication_synchro_quorum = 3/2 + 1 = 2 +match = 'update replication_synchro_quorum = 2' +test_run:wait_log('default', match, nil, 5) + +test_run:cmd('create server replica3 with rpl_master=default,\ + script="replication/replica-quorum-3.lua"') +test_run:cmd('start server replica3 with wait=True, wait_load=True') + +-- 3 replicas -> replication_synchro_quorum = 4/2 + 1 = 3 +match = 'update replication_synchro_quorum = 3' +test_run:wait_log('default', match, nil, 5) + +test_run:cmd('create server replica4 with rpl_master=default,\ + script="replication/replica-quorum-4.lua"') +test_run:cmd('start server replica4 with wait=True, wait_load=True') + +-- 4 replicas -> replication_synchro_quorum = 5/2 + 1 = 3 +match = 'update replication_synchro_quorum = 3' +test_run:wait_log('default', match, nil, 5) + +test_run:cmd('create server replica5 with rpl_master=default,\ + script="replication/replica-quorum-5.lua"') +test_run:cmd('start server replica5 with wait=True, wait_load=True') + +test_run:cmd('create server replica6 with rpl_master=default,\ + script="replication/replica-quorum-6.lua"') +test_run:cmd('start server replica6 with wait=True, wait_load=True') + +-- 6 replicas -> replication_synchro_quorum = 7/2 + 1 = 4 +match = 'update replication_synchro_quorum = 4' +test_run:wait_log('default', match, nil, 5) + +-- 5 replicas left, the commit should pass +test_run:cmd('stop server replica1') +test_run:cmd('delete server replica1') +s:insert{2} + +-- 4 replicas left,the commit should pass +test_run:cmd('stop server replica2') +test_run:cmd('delete server replica2') +s:insert{3} + +-- 3 replicas left, the commit should pass +test_run:cmd('stop server replica3') +test_run:cmd('delete server replica3') +s:insert{4} + +-- 2 replicas left, the commit should NOT pass +-- +-- The replication_synchro_timeout set to a small value to not wait +-- for very long for the case where we know the commit should +-- not pass since replicas are stopped. +box.cfg { replication_synchro_timeout = 0.5 } +test_run:cmd('stop server replica4') +s:insert{5} +-- restore it back and retry +test_run:cmd('start server replica4 with wait=True, wait_load=True') +box.cfg { replication_synchro_timeout = 1000 } +s:insert{5} +test_run:cmd('stop server replica4') +test_run:cmd('delete server replica4') + +-- cleanup leftovers + +test_run:cmd('stop server replica5') +test_run:cmd('delete server replica5') + +test_run:cmd('stop server replica6') +test_run:cmd('delete server replica6') + +box.schema.user.revoke('guest', 'replication') diff --git a/test/replication/replica-quorum-1.lua b/test/replication/replica-quorum-1.lua new file mode 120000 index 000000000..da69ac81c --- /dev/null +++ b/test/replication/replica-quorum-1.lua @@ -0,0 +1 @@ +replica.lua \ No newline at end of file diff --git a/test/replication/replica-quorum-2.lua b/test/replication/replica-quorum-2.lua new file mode 120000 index 000000000..da69ac81c --- /dev/null +++ b/test/replication/replica-quorum-2.lua @@ -0,0 +1 @@ +replica.lua \ No newline at end of file diff --git a/test/replication/replica-quorum-3.lua b/test/replication/replica-quorum-3.lua new file mode 120000 index 000000000..da69ac81c --- /dev/null +++ b/test/replication/replica-quorum-3.lua @@ -0,0 +1 @@ +replica.lua \ No newline at end of file diff --git a/test/replication/replica-quorum-4.lua b/test/replication/replica-quorum-4.lua new file mode 120000 index 000000000..da69ac81c --- /dev/null +++ b/test/replication/replica-quorum-4.lua @@ -0,0 +1 @@ +replica.lua \ No newline at end of file diff --git a/test/replication/replica-quorum-5.lua b/test/replication/replica-quorum-5.lua new file mode 120000 index 000000000..da69ac81c --- /dev/null +++ b/test/replication/replica-quorum-5.lua @@ -0,0 +1 @@ +replica.lua \ No newline at end of file diff --git a/test/replication/replica-quorum-6.lua b/test/replication/replica-quorum-6.lua new file mode 120000 index 000000000..da69ac81c --- /dev/null +++ b/test/replication/replica-quorum-6.lua @@ -0,0 +1 @@ +replica.lua \ No newline at end of file -- 2.26.2
next prev parent reply other threads:[~2020-12-22 11:15 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-12-22 11:14 [Tarantool-patches] [PATCH v6 0/5] qsync: evaluate replication_synchro_quorum dynamically Cyrill Gorcunov 2020-12-22 11:14 ` [Tarantool-patches] [PATCH v6 1/5] cfg: add cfg_isnumber helper Cyrill Gorcunov 2020-12-22 11:14 ` [Tarantool-patches] [PATCH v6 2/5] cfg: rework box_check_replication_synchro_quorum Cyrill Gorcunov 2020-12-22 11:14 ` [Tarantool-patches] [PATCH v6 3/5] cfg: support symbolic evaluation of replication_synchro_quorum Cyrill Gorcunov 2020-12-22 11:14 ` [Tarantool-patches] [PATCH v6 4/5] cfg: more precise check for replication_synchro_quorum value Cyrill Gorcunov 2020-12-22 11:14 ` Cyrill Gorcunov [this message] 2020-12-22 13:49 ` [Tarantool-patches] [PATCH v6 0/5] qsync: evaluate replication_synchro_quorum dynamically Vladislav Shpilevoy 2020-12-22 13:53 ` Cyrill Gorcunov 2020-12-22 14:32 ` Cyrill Gorcunov 2020-12-22 17:13 ` Cyrill Gorcunov 2020-12-23 14:14 ` Vladislav Shpilevoy 2020-12-23 15:54 ` Cyrill Gorcunov 2020-12-23 17:52 ` Vladislav Shpilevoy 2020-12-24 13:55 ` Vladislav Shpilevoy
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20201222111408.48368-6-gorcunov@gmail.com \ --to=gorcunov@gmail.com \ --cc=tarantool-patches@dev.tarantool.org \ --cc=v.shpilevoy@tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH v6 5/5] test: add replication/gh-5446-qsync-eval-quorum.test.lua' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox