Tarantool development patches archive
 help / color / mirror / Atom feed
From: "Alexander V. Tikhonov" <avtikhon@tarantool.org>
To: Oleg Koshovetc <okoshovetc@tarantool.org>,
	Kirill Yukhin <kyukhin@tarantool.org>
Cc: tarantool-patches@dev.tarantool.org
Subject: Re: [Tarantool-patches] [PATCH v1] test: add test filter for vinyl tests
Date: Mon, 21 Dec 2020 13:32:16 +0300	[thread overview]
Message-ID: <20201221103216.GA63937@hpalx> (raw)
In-Reply-To: <db6e2b967dd201ff02b7db5b168e8c659671d059.1608105975.git.avtikhon@tarantool.org>

Hi All, also force pushed new checksums for the tests after its update.

On Wed, Dec 16, 2020 at 11:07:39AM +0300, Alexander V. Tikhonov wrote:
> Cc: tarantool-patches@dev.tarantool.org
> 
> Added test-run filter on box.snapshot error message:
> 
>   'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'
> 
> to avoid of printing changing data in results file to be able to use
> its checksums in fragile list of test-run to rerun it as flaky issue.
> 
> Found issues:
> 
>  1) vinyl/deferred_delete.test.lua
>     https://gitlab.com/tarantool/tarantool/-/jobs/913623306#L4552
> 
>   [036] 2020-12-15 19:10:01.996 [16602] coio vy_log.c:2202 E> failed to process vylog record: delete_slice{slice_id=744, }
>   [036] 2020-12-15 19:10:01.996 [16602] main/103/vinyl vy_log.c:2068 E> ER_INVALID_VYLOG_FILE: Invalid VYLOG file: Slice 744 deleted but not registered
> 
>  2) vinyl/gh-4864-stmt-alloc-fail-compact.test.lua
>     https://gitlab.com/tarantool/tarantool/-/jobs/913810422#L4835
> 
>   [052] @@ -56,9 +56,11 @@
>   [052]  --
>   [052]  dump(true)
>   [052]   | ---
>   [052] - | ...
>   [052] -dump()
>   [052] - | ---
>   [052] + | - error: 'Invalid VYLOG file: Slice 253 deleted but not registered'
>   [052] + | ...
> 
>  3) vinyl/misc.test.lua
>     https://gitlab.com/tarantool/tarantool/-/jobs/913727925#L5284
> 
>   [014] @@ -62,14 +62,14 @@
>   [014]  ...
>   [014]  box.snapshot()
>   [014]  ---
>   [014] -- ok
>   [014] +- error: 'Invalid VYLOG file: Slice 1141 deleted but not registered'
>   [014]  ...
> 
>  4) vinyl/quota.test.lua
>     https://gitlab.com/tarantool/tarantool/-/jobs/914016074#L4595
> 
>   [025] 2020-12-15 22:56:50.192 [25576] coio vy_log.c:2202 E> failed to process vylog record: delete_slice{slice_id=522, }
>   [025] 2020-12-15 22:56:50.193 [25576] main/103/vinyl vy_log.c:2068 E> ER_INVALID_VYLOG_FILE: Invalid VYLOG file: Slice 522 deleted but not registered
> 
>  5) vinyl/update_optimize.test.lua
>     https://gitlab.com/tarantool/tarantool/-/jobs/913728098#L2512
> 
>   [051] 2020-12-15 20:18:43.365 [17147] coio vy_log.c:2202 E> failed to process vylog record: delete_slice{slice_id=350, }
>   [051] 2020-12-15 20:18:43.365 [17147] main/103/vinyl vy_log.c:2068 E> ER_INVALID_VYLOG_FILE: Invalid VYLOG file: Slice 350 deleted but not registered
> 
>  6) vinyl/upsert.test.lua
>     https://gitlab.com/tarantool/tarantool/-/jobs/913623510#L6132
> 
>   [008] @@ -441,7 +441,7 @@
>   [008]  -- Mem has DELETE
>   [008]  box.snapshot()
>   [008]  ---
>   [008] -- ok
>   [008] +- error: 'Invalid VYLOG file: Slice 1411 deleted but not registered'
>   [008]  ...
> 
>  7) vinyl/replica_quota.test.lua
>     https://gitlab.com/tarantool/tarantool/-/jobs/914272656#L5739
> 
>   [023] @@ -41,7 +41,7 @@
>   [023]  ...
>   [023]  box.snapshot()
>   [023]  ---
>   [023] -- ok
>   [023] +- error: 'Invalid VYLOG file: Slice 232 deleted but not registered'
>   [023]  ...
> 
>  8) vinyl/ddl.test.lua
>     https://gitlab.com/tarantool/tarantool/-/jobs/914309343#L4538
> 
>   [039] @@ -81,7 +81,7 @@
>   [039]  ...
>   [039]  box.snapshot()
>   [039]  ---
>   [039] -- ok
>   [039] +- error: 'Invalid VYLOG file: Slice 206 deleted but not registered'
>   [039]  ...
> ---
> 
> Github: https://github.com/tarantool/tarantool/tree/avtikhon/vinyl-filter
> 
>  test/vinyl/ddl.result                               | 5 +++++
>  test/vinyl/ddl.test.lua                             | 3 +++
>  test/vinyl/deferred_delete.result                   | 5 +++++
>  test/vinyl/deferred_delete.test.lua                 | 3 +++
>  test/vinyl/gh-4864-stmt-alloc-fail-compact.result   | 6 ++++++
>  test/vinyl/gh-4864-stmt-alloc-fail-compact.test.lua | 3 +++
>  test/vinyl/misc.result                              | 5 +++++
>  test/vinyl/misc.test.lua                            | 3 +++
>  test/vinyl/quota.result                             | 5 +++++
>  test/vinyl/quota.test.lua                           | 4 ++++
>  test/vinyl/replica_quota.result                     | 5 +++++
>  test/vinyl/replica_quota.test.lua                   | 3 +++
>  test/vinyl/update_optimize.result                   | 5 +++++
>  test/vinyl/update_optimize.test.lua                 | 3 +++
>  test/vinyl/upsert.result                            | 5 +++++
>  test/vinyl/upsert.test.lua                          | 3 +++
>  16 files changed, 66 insertions(+)
> 
> diff --git a/test/vinyl/ddl.result b/test/vinyl/ddl.result
> index 9c453a007..00d76d713 100644
> --- a/test/vinyl/ddl.result
> +++ b/test/vinyl/ddl.result
> @@ -4,6 +4,11 @@ fiber = require('fiber')
>  test_run = require('test_run').new()
>  ---
>  ...
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +---
> +- true
> +...
>  -- sanity checks
>  space = box.schema.space.create('test', {engine = 'vinyl' })
>  ---
> diff --git a/test/vinyl/ddl.test.lua b/test/vinyl/ddl.test.lua
> index 010ec6d79..6be674638 100644
> --- a/test/vinyl/ddl.test.lua
> +++ b/test/vinyl/ddl.test.lua
> @@ -1,6 +1,9 @@
>  fiber = require('fiber')
>  test_run = require('test_run').new()
>  
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +
>  -- sanity checks
>  space = box.schema.space.create('test', {engine = 'vinyl' })
>  space:create_index('pk', {page_size = 0})
> diff --git a/test/vinyl/deferred_delete.result b/test/vinyl/deferred_delete.result
> index 23c93f0f5..5553a359b 100644
> --- a/test/vinyl/deferred_delete.result
> +++ b/test/vinyl/deferred_delete.result
> @@ -4,6 +4,11 @@ test_run = require('test_run').new()
>  fiber = require('fiber')
>  ---
>  ...
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +---
> +- true
> +...
>  --
>  -- Create a space with secondary indexes and check that REPLACE and
>  -- DELETE requests do not look up the old tuple in the primary index
> diff --git a/test/vinyl/deferred_delete.test.lua b/test/vinyl/deferred_delete.test.lua
> index 1bce954c5..b1f464adf 100644
> --- a/test/vinyl/deferred_delete.test.lua
> +++ b/test/vinyl/deferred_delete.test.lua
> @@ -1,6 +1,9 @@
>  test_run = require('test_run').new()
>  fiber = require('fiber')
>  
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +
>  --
>  -- Create a space with secondary indexes and check that REPLACE and
>  -- DELETE requests do not look up the old tuple in the primary index
> diff --git a/test/vinyl/gh-4864-stmt-alloc-fail-compact.result b/test/vinyl/gh-4864-stmt-alloc-fail-compact.result
> index 6a1f49626..93c4c80e4 100644
> --- a/test/vinyl/gh-4864-stmt-alloc-fail-compact.result
> +++ b/test/vinyl/gh-4864-stmt-alloc-fail-compact.result
> @@ -9,6 +9,12 @@ digest = require('digest')
>   | ---
>   | ...
>  
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> + | ---
> + | - true
> + | ...
> +
>  s = box.schema.space.create('test', {engine = 'vinyl'})
>   | ---
>   | ...
> diff --git a/test/vinyl/gh-4864-stmt-alloc-fail-compact.test.lua b/test/vinyl/gh-4864-stmt-alloc-fail-compact.test.lua
> index 4b3c55505..1482abecf 100644
> --- a/test/vinyl/gh-4864-stmt-alloc-fail-compact.test.lua
> +++ b/test/vinyl/gh-4864-stmt-alloc-fail-compact.test.lua
> @@ -2,6 +2,9 @@ test_run = require('test_run').new()
>  fiber = require('fiber')
>  digest = require('digest')
>  
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +
>  s = box.schema.space.create('test', {engine = 'vinyl'})
>  _ = s:create_index('pk', {run_count_per_level = 100, page_size = 128, range_size = 1024})
>  
> diff --git a/test/vinyl/misc.result b/test/vinyl/misc.result
> index e647b93c3..15d154854 100644
> --- a/test/vinyl/misc.result
> +++ b/test/vinyl/misc.result
> @@ -4,6 +4,11 @@ test_run = require('test_run').new()
>  fiber = require('fiber')
>  ---
>  ...
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +---
> +- true
> +...
>  --
>  -- gh-2784: do not validate space formatted but not indexed fields
>  -- in surrogate statements.
> diff --git a/test/vinyl/misc.test.lua b/test/vinyl/misc.test.lua
> index 0a7c7fc99..66bf88f36 100644
> --- a/test/vinyl/misc.test.lua
> +++ b/test/vinyl/misc.test.lua
> @@ -1,6 +1,9 @@
>  test_run = require('test_run').new()
>  fiber = require('fiber')
>  
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +
>  --
>  -- gh-2784: do not validate space formatted but not indexed fields
>  -- in surrogate statements.
> diff --git a/test/vinyl/quota.result b/test/vinyl/quota.result
> index 940df4e49..70e81453d 100644
> --- a/test/vinyl/quota.result
> +++ b/test/vinyl/quota.result
> @@ -4,6 +4,11 @@ env = require('test_run')
>  test_run = env.new()
>  ---
>  ...
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +---
> +- true
> +...
>  --
>  -- Restart the server because need to reset quota used memory
>  -- after previous tests.
> diff --git a/test/vinyl/quota.test.lua b/test/vinyl/quota.test.lua
> index a2793a015..081c573f3 100644
> --- a/test/vinyl/quota.test.lua
> +++ b/test/vinyl/quota.test.lua
> @@ -1,6 +1,10 @@
>  env = require('test_run')
>  test_run = env.new()
>  
> +
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +
>  --
>  -- Restart the server because need to reset quota used memory
>  -- after previous tests.
> diff --git a/test/vinyl/replica_quota.result b/test/vinyl/replica_quota.result
> index bd09e764d..f7ab2482d 100644
> --- a/test/vinyl/replica_quota.result
> +++ b/test/vinyl/replica_quota.result
> @@ -1,6 +1,11 @@
>  test_run = require('test_run').new()
>  ---
>  ...
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +---
> +- true
> +...
>  box.schema.user.grant('guest', 'replication')
>  ---
>  ...
> diff --git a/test/vinyl/replica_quota.test.lua b/test/vinyl/replica_quota.test.lua
> index 1f373fd47..b471f623e 100644
> --- a/test/vinyl/replica_quota.test.lua
> +++ b/test/vinyl/replica_quota.test.lua
> @@ -1,5 +1,8 @@
>  test_run = require('test_run').new()
>  
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +
>  box.schema.user.grant('guest', 'replication')
>  
>  s = box.schema.space.create('test', { engine = 'vinyl' })
> diff --git a/test/vinyl/update_optimize.result b/test/vinyl/update_optimize.result
> index 09370e7d5..01ff7e043 100644
> --- a/test/vinyl/update_optimize.result
> +++ b/test/vinyl/update_optimize.result
> @@ -6,6 +6,11 @@ test_run:cmd('restart server default')
>  fiber = require('fiber')
>  ---
>  ...
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +---
> +- true
> +...
>  -- optimize one index
>  space = box.schema.space.create('test', { engine = 'vinyl' })
>  ---
> diff --git a/test/vinyl/update_optimize.test.lua b/test/vinyl/update_optimize.test.lua
> index a0de6e4cd..d7556560e 100644
> --- a/test/vinyl/update_optimize.test.lua
> +++ b/test/vinyl/update_optimize.test.lua
> @@ -3,6 +3,9 @@ test_run = require('test_run').new()
>  test_run:cmd('restart server default')
>  fiber = require('fiber')
>  
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +
>  -- optimize one index
>  
>  space = box.schema.space.create('test', { engine = 'vinyl' })
> diff --git a/test/vinyl/upsert.result b/test/vinyl/upsert.result
> index fe673ad6f..a935e0911 100644
> --- a/test/vinyl/upsert.result
> +++ b/test/vinyl/upsert.result
> @@ -1,6 +1,11 @@
>  test_run = require('test_run').new()
>  ---
>  ...
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +---
> +- true
> +...
>  -- gh-1671 upsert is broken in a transaction
>  -- upsert after upsert
>  space = box.schema.space.create('test', { engine = 'vinyl' })
> diff --git a/test/vinyl/upsert.test.lua b/test/vinyl/upsert.test.lua
> index b62c19978..36b0f5ec8 100644
> --- a/test/vinyl/upsert.test.lua
> +++ b/test/vinyl/upsert.test.lua
> @@ -1,5 +1,8 @@
>  test_run = require('test_run').new()
>  
> +test_run:cmd("push filter 'Invalid VYLOG file: Slice [0-9]+ deleted but not registered'" .. \
> +             " to 'Invalid VYLOG file: Slice <NUM> deleted but not registered'")
> +
>  -- gh-1671 upsert is broken in a transaction
>  
>  -- upsert after upsert
> -- 
> 2.25.1
> 

  reply	other threads:[~2020-12-21 10:32 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-16  8:07 Alexander V. Tikhonov
2020-12-21 10:32 ` Alexander V. Tikhonov [this message]
2020-12-21 10:35   ` Oleg Koshovetc
2020-12-22  9:30 ` Kirill Yukhin

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=20201221103216.GA63937@hpalx \
    --to=avtikhon@tarantool.org \
    --cc=kyukhin@tarantool.org \
    --cc=okoshovetc@tarantool.org \
    --cc=tarantool-patches@dev.tarantool.org \
    --subject='Re: [Tarantool-patches] [PATCH v1] test: add test filter for vinyl 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