From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp48.i.mail.ru (smtp48.i.mail.ru [94.100.177.108]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 3DC95469719 for ; Fri, 13 Nov 2020 17:05:37 +0300 (MSK) References: <508f66c80ca243afd1a0044bda250ae3cb618c32.1605170394.git.sergeyb@tarantool.org> From: Serge Petrenko Message-ID: <426699a5-18e2-8b95-2620-7108260f9dac@tarantool.org> Date: Fri, 13 Nov 2020 17:05:36 +0300 MIME-Version: 1.0 In-Reply-To: <508f66c80ca243afd1a0044bda250ae3cb618c32.1605170394.git.sergeyb@tarantool.org> Content-Type: text/plain; charset="utf-8"; format="flowed" Content-Transfer-Encoding: 8bit Content-Language: en-GB Subject: Re: [Tarantool-patches] [PATCH 3/3 v2] replication: add test with change space sync mode in a loop List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: sergeyb@tarantool.org, tarantool-patches@dev.tarantool.org, v.shpilevoy@tarantool.org 12.11.2020 12:11, sergeyb@tarantool.org пишет: > From: Sergey Bronnikov > > Closes #5055 > Part of #5144 > --- Thanks for the patch! LGTM. > test/replication/qsync_sync_mode.result | 114 ++++++++++++++++++++++ > test/replication/qsync_sync_mode.test.lua | 56 +++++++++++ > 2 files changed, 170 insertions(+) > create mode 100644 test/replication/qsync_sync_mode.result > create mode 100644 test/replication/qsync_sync_mode.test.lua > > diff --git a/test/replication/qsync_sync_mode.result b/test/replication/qsync_sync_mode.result > new file mode 100644 > index 000000000..66f6a70b2 > --- /dev/null > +++ b/test/replication/qsync_sync_mode.result > @@ -0,0 +1,114 @@ > +-- test-run result file version 2 > +env = require('test_run') > + | --- > + | ... > +test_run = env.new() > + | --- > + | ... > +engine = test_run:get_cfg('engine') > + | --- > + | ... > +fiber = require('fiber') > + | --- > + | ... > +math = require('math') > + | --- > + | ... > + > +orig_synchro_quorum = box.cfg.replication_synchro_quorum > + | --- > + | ... > +orig_synchro_timeout = box.cfg.replication_synchro_timeout > + | --- > + | ... > + > +math.randomseed(os.time()) > + | --- > + | ... > +random_boolean = function() \ > + if (math.random(1, 10) > 5) then \ > + return true \ > + else \ > + return false \ > + end \ > +end > + | --- > + | ... > + > +box.schema.user.grant('guest', 'replication') > + | --- > + | ... > + > +-- Setup an cluster with two instances. > +test_run:cmd('create server replica with rpl_master=default,\ > + script="replication/replica.lua"') > + | --- > + | - true > + | ... > +test_run:cmd('start server replica with wait=True, wait_load=True') > + | --- > + | - true > + | ... > + > +-- Write data to a leader and enable/disable sync mode in a loop. > +-- Testcase setup. > +_ = box.schema.space.create('sync', {is_sync=true, engine=engine}) > + | --- > + | ... > +_ = box.space.sync:create_index('pk') > + | --- > + | ... > +box.cfg{replication_synchro_quorum=2, replication_synchro_timeout=1000} > + | --- > + | ... > + > +-- Testcase body. > +for i = 1,100 do \ > + box.space.sync:alter{is_sync=random_boolean()} \ > + box.space.sync:insert{i} \ > + test_run:switch('replica') \ > + test_run:wait_cond(function() return box.space.sync:get{i} ~= nil end) \ > + test_run:switch('default') \ > + test_run:wait_cond(function() return box.space.sync:get{i} ~= nil end) \ > +end > + | --- > + | ... > +box.space.sync:count() -- 100 > + | --- > + | - 100 > + | ... > + > +-- Testcase cleanup. > +test_run:switch('default') > + | --- > + | - true > + | ... > +box.space.sync:drop() > + | --- > + | ... > + > +-- Teardown. > +test_run:cmd('switch default') > + | --- > + | - true > + | ... > +test_run:cmd('stop server replica') > + | --- > + | - true > + | ... > +test_run:cmd('delete server replica') > + | --- > + | - true > + | ... > +test_run:cleanup_cluster() > + | --- > + | ... > +box.schema.user.revoke('guest', 'replication') > + | --- > + | ... > +box.cfg{ \ > + replication_synchro_quorum = orig_synchro_quorum, \ > + replication_synchro_timeout = orig_synchro_timeout, \ > +} > + | --- > + | ... > diff --git a/test/replication/qsync_sync_mode.test.lua b/test/replication/qsync_sync_mode.test.lua > new file mode 100644 > index 000000000..ea5725169 > --- /dev/null > +++ b/test/replication/qsync_sync_mode.test.lua > @@ -0,0 +1,56 @@ > +env = require('test_run') > +test_run = env.new() > +engine = test_run:get_cfg('engine') > +fiber = require('fiber') > +math = require('math') > + > +orig_synchro_quorum = box.cfg.replication_synchro_quorum > +orig_synchro_timeout = box.cfg.replication_synchro_timeout > + > +math.randomseed(os.time()) > +random_boolean = function() \ > + if (math.random(1, 10) > 5) then \ > + return true \ > + else \ > + return false \ > + end \ > +end > + > +box.schema.user.grant('guest', 'replication') > + > +-- Setup an cluster with two instances. > +test_run:cmd('create server replica with rpl_master=default,\ > + script="replication/replica.lua"') > +test_run:cmd('start server replica with wait=True, wait_load=True') > + > +-- Write data to a leader and enable/disable sync mode in a loop. > +-- Testcase setup. > +_ = box.schema.space.create('sync', {is_sync=true, engine=engine}) > +_ = box.space.sync:create_index('pk') > +box.cfg{replication_synchro_quorum=2, replication_synchro_timeout=1000} > + > +-- Testcase body. > +for i = 1,100 do \ > + box.space.sync:alter{is_sync=random_boolean()} \ > + box.space.sync:insert{i} \ > + test_run:switch('replica') \ > + test_run:wait_cond(function() return box.space.sync:get{i} ~= nil end) \ > + test_run:switch('default') \ > + test_run:wait_cond(function() return box.space.sync:get{i} ~= nil end) \ > +end > +box.space.sync:count() -- 100 > + > +-- Testcase cleanup. > +test_run:switch('default') > +box.space.sync:drop() > + > +-- Teardown. > +test_run:cmd('switch default') > +test_run:cmd('stop server replica') > +test_run:cmd('delete server replica') > +test_run:cleanup_cluster() > +box.schema.user.revoke('guest', 'replication') > +box.cfg{ \ > + replication_synchro_quorum = orig_synchro_quorum, \ > + replication_synchro_timeout = orig_synchro_timeout, \ > +} -- Serge Petrenko