From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id 13349298F0 for ; Sun, 24 Mar 2019 01:41:59 -0400 (EDT) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id FXZZ2Mw0mhBD for ; Sun, 24 Mar 2019 01:41:58 -0400 (EDT) Received: from smtpng2.m.smailru.net (smtpng2.m.smailru.net [94.100.179.3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id 4B70E298DF for ; Sun, 24 Mar 2019 01:41:58 -0400 (EDT) Date: Sun, 24 Mar 2019 08:41:51 +0300 From: Alexander Turenko Subject: [tarantool-patches] Re: [PATCH v2 2/2] test: added loop with restart for fullmesh routine Message-ID: <20190324054150.57crfwhsfursfw66@tkn_work_nb> References: <6726cdd338a2c24005e72ceec9a6badc072bcdbd.1553148177.git.avtikhon@tarantool.org> <463a0c7cf85bf52ce80a27ac356a850ac789e442.1553148177.git.avtikhon@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <463a0c7cf85bf52ce80a27ac356a850ac789e442.1553148177.git.avtikhon@tarantool.org> Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-Help: List-Unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-Subscribe: List-Owner: List-post: List-Archive: To: "Alexander V. Tikhonov" Cc: tarantool-patches@freelists.org, Georgy Kirichenko I created more or less minimal test case for the problem on the branch Totktonada/reproduce-create-cluster-fail (./run.sh reproduces the problem). Hope Georgy will able to pay some time to look into (CCed him). I think that we should not hide non-investigated problems in such way except, maybe, really urgent cases, only as temporary workaround and with a reference to an issue about a real problem (to don't forget to revert after a fix). If we'll decide to do that retries we should do it in test_run.lua (I guess in create_cluster()). Let's wait for an initial investigation from Georgy or one of other developers. WBR, Alexander Turenko. On Thu, Mar 21, 2019 at 09:03:06AM +0300, Alexander V. Tikhonov wrote: > Add loop with restart for fullmesh routine. > --- > test/replication/autobootstrap.result | 3 ++- > test/replication/autobootstrap.test.lua | 2 +- > test/replication/autobootstrap_guest.result | 2 +- > test/replication/autobootstrap_guest.test.lua | 2 +- > test/replication/before_replace.result | 2 +- > test/replication/before_replace.test.lua | 2 +- > test/replication/ddl.result | 2 +- > test/replication/ddl.test.lua | 2 +- > test/replication/misc.result | 2 +- > test/replication/misc.test.lua | 2 +- > test/replication/on_replace.result | 2 +- > test/replication/on_replace.test.lua | 2 +- > test/replication/quorum.result | 4 ++-- > test/replication/quorum.test.lua | 4 ++-- > test/replication/rebootstrap.result | 2 +- > test/replication/rebootstrap.test.lua | 2 +- > test/replication/recover_missing_xlog.result | 9 ++------- > test/replication/recover_missing_xlog.test.lua | 7 ++----- > test/replication/replicaset_ro_mostly.result | 1 + > test/replication/show_error_on_disconnect.result | 2 +- > test/replication/show_error_on_disconnect.test.lua | 2 +- > 21 files changed, 26 insertions(+), 32 deletions(-)