<HTML><BODY>Build: https://travis-ci.org/tarantool/tarantool/builds/455965753<br><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">
Пятница, 16 ноября 2018, 12:27 +03:00 от Alexander Turenko <alexander.turenko@tarantool.org>:<br><br><div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_15423604510000000836_BODY">Please, investigate errors that were arise in CI:<br><br>
> TypeError: cleanup() got multiple values for keyword argument 'dirname'<br><br><a href="https://travis-ci.org/tarantool/tarantool/jobs/455877309#L2652" target="_blank">https://travis-ci.org/tarantool/tarantool/jobs/455877309#L2652</a><br><a href="https://travis-ci.org/tarantool/tarantool/jobs/455877310#L2730" target="_blank">https://travis-ci.org/tarantool/tarantool/jobs/455877310#L2730</a><br data-mce-bogus="1"></div></div></div></div></blockquote>Fixed:<br>diff --git a/test/replication-py/cluster.test.py b/test/replication-py/cluster.test.py<br>index 14598b798..b5f59f40f 100644<br>--- a/test/replication-py/cluster.test.py<br>+++ b/test/replication-py/cluster.test.py<br>@@ -209,7 +209,7 @@ master.admin('box.info.vclock[%d] == 2' % replica_id)<br> <br> master.admin("box.cfg{ replication = '' }")<br> replica.stop()<br>-replica.cleanup(True)<br>+replica.cleanup()<br> <br> print '-------------------------------------------------------------'<br> print 'Start a new replica and check that server_id, LSN is re-used'<br>@@ -238,7 +238,7 @@ master.admin('box.info.vclock[%d] == 2' % replica_id)<br> replica.admin('box.info.vclock[%d] == 2' % replica_id)<br> <br> replica.stop()<br>-replica.cleanup(True)<br>+replica.cleanup()<br> <br> print '-------------------------------------------------------------'<br> print 'JOIN replica to read-only master'<br>diff --git a/test/replication-py/conflict.test.py b/test/replication-py/conflict.test.py<br>index 41c6fca73..f40148318 100644<br>--- a/test/replication-py/conflict.test.py<br>+++ b/test/replication-py/conflict.test.py<br>@@ -43,12 +43,12 @@ def parallel_run(cmd1, cmd2, compare):<br> def prepare_cluster():<br> print 'reset master-master replication'<br> master.stop()<br>- master.cleanup(True)<br>+ master.cleanup()<br> master.start()<br> master.admin("box.schema.user.grant('guest', 'replication')", silent=True)<br> <br> replica.stop()<br>- replica.cleanup(True)<br>+ replica.cleanup()<br> replica.start()<br> <br> master.admin("box.cfg{replication='%s'}" % replica.iproto.uri, silent=True)<br>@@ -114,7 +114,7 @@ check_replication([master, replica])<br> <br> # cleanup<br> replica.stop()<br>-replica.cleanup(True)<br>+replica.cleanup()<br> server.stop()<br>-server.cleanup(True)<br>+server.cleanup()<br> server.deploy()<br>diff --git a/test/replication-py/init_storage.test.py b/test/replication-py/init_storage.test.py<br>index 32b4639f1..4be531f8d 100644<br>--- a/test/replication-py/init_storage.test.py<br>+++ b/test/replication-py/init_storage.test.py<br>@@ -26,7 +26,7 @@ replica.admin('box.space.test:select()')<br> replica.restart()<br> replica.admin('box.space.test:select()')<br> replica.stop()<br>-replica.cleanup(True)<br>+replica.cleanup()<br> <br> print '-------------------------------------------------------------'<br> print 'replica test 2 (must be ok)'<br>@@ -49,7 +49,7 @@ for i in range(1, 20):<br> replica.admin('space:get{%d}' % i)<br> <br> replica.stop()<br>-replica.cleanup(True)<br>+replica.cleanup()<br> <br> print '-------------------------------------------------------------'<br> print 'reconnect on JOIN/SUBSCRIBE'<br>@@ -77,7 +77,7 @@ replica.wait_until_started()<br> print 'ok'<br> <br> replica.stop()<br>-replica.cleanup(True)<br>+replica.cleanup()<br> <br> server.stop()<br> server.deploy()<br>diff --git a/test/replication-py/swap.test.py b/test/replication-py/swap.test.py<br>index b18940517..98eeeea6d 100644<br>--- a/test/replication-py/swap.test.py<br>+++ b/test/replication-py/swap.test.py<br>@@ -119,6 +119,6 @@ for i in range(REPEAT):<br> <br> # Cleanup.<br> replica.stop()<br>-replica.cleanup(True)<br>+replica.cleanup()<br> server.stop()<br> server.deploy()<br><br><br><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_15423604510000000836_BODY"><br><br>
> RuntimeError: Broken tarantool console handshake<br><br><a href="https://travis-ci.org/tarantool/tarantool/jobs/455877312#L2655" target="_blank">https://travis-ci.org/tarantool/tarantool/jobs/455877312#L2655</a><br></div></div></div></div></blockquote>Looks like this error is not related.<br><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;"><div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_15423604510000000836_BODY"><br>
WBR, Alexander Turenko.<br><br>
On Fri, Nov 16, 2018 at 11:58:36AM +0300, Sergei Voronezhskii wrote:<br>
> * Perform the clean up if param `pre_cleanup = True` in suite.ini.<br>
> It cleans checkpoints in workdir for tap tests. And for app tests<br>
> cleans spaces/users/roles/globals/functions and checks count of<br>
> `box.space._cluster`<br>
> <br>
> Fixes #1961<br>
> ---<br>
> BRANCH: <a href="https://github.com/tarantool/tarantool/tree/sergw/update-test-run" target="_blank">https://github.com/tarantool/tarantool/tree/sergw/update-test-run</a><br>
> ISSUE: <a href="https://github.com/tarantool/tarantool/issues/1961" target="_blank">https://github.com/tarantool/tarantool/issues/1961</a><br>
> PR: <a href="https://github.com/tarantool/test-run/pull/124" target="_blank">https://github.com/tarantool/test-run/pull/124</a><br>
> test-run | 2 +-<br>
> 1 file changed, 1 insertion(+), 1 deletion(-)<br>
> <br>
> diff --git a/test-run b/test-run<br>
> index 670f330aa..d804b6a6e 160000<br>
> --- a/test-run<br>
> +++ b/test-run<br>
> @@ -1 +1 @@<br>
> -Subproject commit 670f330aacaf44bc8b1f969fa0cd5f811c5ceb1b<br>
> +Subproject commit d804b6a6ead6fec466622435fddf2218966d7c06<br>
> -- <br>
> 2.18.0<br>
> <br></div></div></div></div></blockquote>
<br>
<br>-- <br>Sergei Voronezhskii<br></BODY></HTML>