Tarantool development patches archive
 help / color / mirror / Atom feed
From: sergeyb@tarantool.org
To: tarantool-patches@dev.tarantool.org, v.shpilevoy@tarantool.org
Subject: [Tarantool-patches] [PATCH 0/4 v3] Additional qsync tests
Date: Tue, 17 Nov 2020 19:13:47 +0300	[thread overview]
Message-ID: <cover.1605629206.git.sergeyb@tarantool.org> (raw)

From: Sergey Bronnikov <sergeyb@tarantool.org>

Changes v3:

- added negative case with 'broken' quorum in test with random leader
- reduced a number of iterations from 200 to 30 in test with random leader
- moved testcase with box.ctl.clear_synchro_queue() on unconfigured box from
  replication/qsync_basic.test.lua to app-tap/cfg.test.lua (added as separate
  patch).
- added wait_cond() to get final box.space.sync:count()

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/217357503

1. https://lists.tarantool.org/pipermail/tarantool-patches/2020-September/019227.html

Sergey Bronnikov (4):
  replication: run clear_synchro_queue() with unconfigured box
  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/app-tap/cfg.test.lua                     |   8 +-
 test/replication/qsync.lua                    |  31 ++++
 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           | 129 +++++++++++++++
 test/replication/qsync_basic.test.lua         |  45 ++++++
 test/replication/qsync_random_leader.result   | 148 ++++++++++++++++++
 test/replication/qsync_random_leader.test.lua |  76 +++++++++
 test/replication/qsync_sync_mode.result       | 114 ++++++++++++++
 test/replication/qsync_sync_mode.test.lua     |  56 +++++++
 13 files changed, 611 insertions(+), 1 deletion(-)
 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

             reply	other threads:[~2020-11-17 16:13 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-17 16:13 sergeyb [this message]
2020-11-17 16:13 ` [Tarantool-patches] [PATCH 1/4] replication: run clear_synchro_queue() with unconfigured box sergeyb
2020-11-21 14:41   ` Vladislav Shpilevoy
2020-11-23 14:44     ` Sergey Bronnikov
2020-11-17 16:13 ` [Tarantool-patches] [PATCH 2/4] replication: test clear_synchro_queue() function sergeyb
2020-11-21 14:41   ` Vladislav Shpilevoy
2020-11-23 15:13     ` Sergey Bronnikov
2020-11-17 16:13 ` [Tarantool-patches] [PATCH 3/4] replication: add test with random leaders promotion and demotion sergeyb
2020-11-21 14:41   ` Vladislav Shpilevoy
2020-11-24  8:39     ` Sergey Bronnikov
2020-11-25 22:02       ` Vladislav Shpilevoy
2020-11-17 16:13 ` [Tarantool-patches] [PATCH 4/4] replication: add test with change space sync mode in a loop sergeyb
2020-11-21 14:41   ` Vladislav Shpilevoy
2020-11-23 15:42     ` Sergey Bronnikov
2020-11-25 22:02   ` 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=cover.1605629206.git.sergeyb@tarantool.org \
    --to=sergeyb@tarantool.org \
    --cc=tarantool-patches@dev.tarantool.org \
    --cc=v.shpilevoy@tarantool.org \
    --subject='Re: [Tarantool-patches] [PATCH 0/4 v3] 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