tarantool / tarantool / 13591079473
87%
master: 88%

LAST BUILD BRANCH: ninarodicova/test5
DEFAULT BRANCH: master
Ran
Jobs 1
Files 514
Run time 26min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 87.372% (-0.03%) from 87.397%
13591079473

push

github

locker 
test/vinyl/select_consistency: make garbage check more robust

The test fails now and then on CI with:

```
[035] vinyl-luatest/select_consistency_test.lua                     [ fail ]
[035] Test failed! Output from reject file /tmp/t/rejects/vinyl-luatest/select_consistency.reject:
[035] Tarantool version is 2.11.7-entrypoint-10-g130294905b
[035] TAP version 13
[035] 1..2
[035] # Started on Thu Feb 27 09:21:34 2025
[035] # Starting group: vinyl.select_consistency.defer_deletes:false
[035] ok     1	vinyl.select_consistency.defer_deletes:false.test_select_consistency
[035] # Starting group: vinyl.select_consistency.defer_deletes:true
[035] not ok 2	vinyl.select_consistency.defer_deletes:true.test_select_consistency
[035] #   ...tarantool/test/vinyl-luatest/select_consistency_test.lua:296: expected: 675, actual: 1094
[035] #   stack traceback:
[035] #   	...tarantool/test/vinyl-luatest/select_consistency_test.lua:296: in function <...tarantool/test/vinyl-luatest/select_consistency_test.lua:273>
[035] #   	...tarantool/test/vinyl-luatest/select_consistency_test.lua:273: in function 'vinyl.select_consistency.defer_deletes:true.test_select_consistency'
[035] #   artifacts:
[035] #   	master -> /tmp/t/035_vinyl-luatest/artifacts/master-Fp1zE14Fq-Dr
[035] # Ran 2 tests in 12.841 seconds, 1 succeeded, 1 failed
```

The test fails while checking that there's no garbage statements in
the test space. We already fixed this failure in the scope of #11019:
the reason was that `index:compact()` didn't work if it raced with range
splitting. Now, the problem is different and it's in the test itself.

First, there may be compaction tasks still running after the test is
complete. We should wait for them before forcing major compaction and
checking the counters, otherwise major compaction may be skipped.

Second, a single run file may contain garbage statements that were
not compacted because they were visible from some read view. Since
`index:compact()` doesn't trigger ... (continued)

68819 of 122059 branches covered (56.38%)

101489 of 116157 relevant lines covered (87.37%)

1947216.7 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
1
95.89
 -0.05% src/box/iproto.cc
1
85.48
 -1.61% src/box/lua/ctl.c
1
95.36
 -0.31% src/box/raft.c
1
80.74
 -0.41% src/box/tuple_bloom.c
1
96.55
 -1.15% src/histogram.c
1
90.38
 -1.92% src/lib/core/trigger.cc
1
91.34
 -0.11% src/lib/swim/swim.c
1
97.41
 -0.83% src/lib/vclock/vclock.c
2
90.69
 -0.07% src/box/box.cc
2
92.2
 0.0% src/box/vinyl.c
2
84.34
 -0.14% src/box/xlog.c
2
85.06
 -0.32% src/lib/core/say.c
2
95.45
 -4.55% src/lib/json/json.h
2
94.11
 -0.39% src/lua/fiber.c
3
90.9
 -0.31% src/box/vy_scheduler.c
3
92.93
 -0.27% src/lib/core/fiber.c
4
94.25
 -0.8% src/box/xrow_update_field.c
5
87.02
 -0.28% src/box/vy_run.c
9
60.42
 -4.03% src/lib/core/fio.c
Jobs
ID Job ID Ran Files Coverage
1 13591079473.1 514
87.37
 GitHub Action Run
Source Files on build 13591079473