From: sergeyb@tarantool.org
To: tarantool-patches@dev.tarantool.org, v.shpilevoy@tarantool.org,
sergepetrenko@tarantool.org
Subject: [Tarantool-patches] [PATCH 0/3 v2] Additional qsync tests
Date: Thu, 12 Nov 2020 12:10:55 +0300 [thread overview]
Message-ID: <cover.1605170394.git.sergeyb@tarantool.org> (raw)
From: Sergey Bronnikov <sergeyb@tarantool.org>
Changes v2:
- removed negative case in test with random leader: no more stuck tx's in a
limbo before switching a leader. I spend couple of hours to get it work, but
failed. Some details: to avoid block a loop I should execute tx in a fiber when
quorum is set to 'broken', but both test_run:eval() and insert via netbox
connection in a fiber haven't insert value even when fiber is dead.
- check inserted values in test with random leader as Sergey asked on previous
review
- removed negative case in test with random sync/async mode, Sergey said it is
pointless [1]
- added test with change enable/disable sync mode, it has LGTM from Sergey [1]
- reduced a cluster size in test with random leader from 32 to 5 instances as
Vlad asked on previous review
- aligned tests with 80 symbols per line
- adjusted values of synchro timeouts in box.cfg to make tests more reliable
GH branch: ligurio/gh-4842-qsync-testing
Github issue: https://github.com/tarantool/tarantool/issues/5055
Gitlab CI: https://gitlab.com/tarantool/tarantool/-/pipelines/215091265
1. https://lists.tarantool.org/pipermail/tarantool-patches/2020-September/019227.html
Sergey Bronnikov (3):
replication: test clear_synchro_queue() function
replication: add test with random leaders promotion and demotion
replication: add test with change space sync mode in a loop
test/replication/qsync.lua | 30 ++++
test/replication/qsync1.lua | 1 +
test/replication/qsync2.lua | 1 +
test/replication/qsync3.lua | 1 +
test/replication/qsync4.lua | 1 +
test/replication/qsync5.lua | 1 +
test/replication/qsync_basic.result | 136 +++++++++++++++++
test/replication/qsync_basic.test.lua | 50 +++++++
test/replication/qsync_random_leader.result | 139 ++++++++++++++++++
test/replication/qsync_random_leader.test.lua | 68 +++++++++
test/replication/qsync_sync_mode.result | 114 ++++++++++++++
test/replication/qsync_sync_mode.test.lua | 56 +++++++
12 files changed, 598 insertions(+)
create mode 100644 test/replication/qsync.lua
create mode 120000 test/replication/qsync1.lua
create mode 120000 test/replication/qsync2.lua
create mode 120000 test/replication/qsync3.lua
create mode 120000 test/replication/qsync4.lua
create mode 120000 test/replication/qsync5.lua
create mode 100644 test/replication/qsync_random_leader.result
create mode 100644 test/replication/qsync_random_leader.test.lua
create mode 100644 test/replication/qsync_sync_mode.result
create mode 100644 test/replication/qsync_sync_mode.test.lua
--
2.25.1
next reply other threads:[~2020-11-12 9:11 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-12 9:10 sergeyb [this message]
2020-11-12 9:10 ` [Tarantool-patches] [PATCH 1/3 v2] replication: test clear_synchro_queue() function sergeyb
2020-11-13 13:12 ` Serge Petrenko
2020-11-16 11:11 ` Sergey Bronnikov
2020-11-16 14:44 ` Serge Petrenko
2020-11-12 9:10 ` [Tarantool-patches] [PATCH 2/3 v2] replication: add test with random leaders promotion and demotion sergeyb
2020-11-13 15:10 ` Serge Petrenko
2020-11-16 9:10 ` Sergey Bronnikov
2020-11-16 14:00 ` Sergey Bronnikov
2020-11-16 14:48 ` Serge Petrenko
2020-11-16 14:53 ` Serge Petrenko
2020-11-16 16:04 ` Sergey Bronnikov
2020-11-12 9:11 ` [Tarantool-patches] [PATCH 3/3 v2] replication: add test with change space sync mode in a loop sergeyb
2020-11-13 14:05 ` Serge Petrenko
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=cover.1605170394.git.sergeyb@tarantool.org \
--to=sergeyb@tarantool.org \
--cc=sergepetrenko@tarantool.org \
--cc=tarantool-patches@dev.tarantool.org \
--cc=v.shpilevoy@tarantool.org \
--subject='Re: [Tarantool-patches] [PATCH 0/3 v2] Additional qsync tests' \
/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