[Tarantool-patches] [PATCH v1] test: add test filter for vinyl tests

Alexander V. Tikhonov avtikhon at tarantool.org
Wed Dec 16 11:07:39 MSK 2020


Cc: tarantool-patches at 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



More information about the Tarantool-patches mailing list