Bug#1043118: victoriametrics: FTBFS: test failure

2023-08-10 Thread Shengjing Zhu 
On Thu, Aug 10, 2023 at 5:57 PM Guillem Jover  wrote:
>
> Hi!
> I cannot reproduce this one on my sid chroot, but another error
> completely different in TestMarshalUnmarshalInt64Array.
>

This is a flaky test, I have reported it to upstream and it has been fixed.
https://github.com/VictoriaMetrics/VictoriaMetrics/issues/3683

-- 
Shengjing Zhu

Bug#1043118: victoriametrics: FTBFS: test failure

2023-08-10 Thread Guillem Jover 
Hi!

On Tue, 2023-08-08 at 14:59:45 +0300, Denys Holius wrote:
> The latest version of VictoriaMetrics v1.92.0
>  should be marked as
> LTS but it has some backwards-incompatible changes. Meanwhile v1.87.6
>  was marked as LTS
> and maybe this version contains bugfix.

I have pending updating to a new release, but when I looked at that
before the bookworm release, that involved packaging a ton of
dependencies (and transitive dependencies), I need to allocate some
time to do that now, but that has not seemed urgent.

> 1. Could you please point me to a documentation which I can use to
>reproduce this bug?

I've covered this (somewhat) in an reply to Denys off-BTS.

> 2. Maybe we can make a bugfix or patch and send a PR to
>https://salsa.debian.org/go-team/packages/victoriametrics ?

Sure, if you do that, I'd do an upload with such fix. Although…

> On 06.08.23 12:16, Sebastian Ramacher wrote:
> > === RUN   TestMergeForciblyStop
> > 2023-08-05T22:21:13.980Zpanic   
> > VictoriaMetrics/lib/storage/inmemory_part.go:37 BUG: Inmemory.InitFromRows 
> > must accept at least one row
> > --- FAIL: TestMergeForciblyStop (0.00s)
> > panic: BUG: Inmemory.InitFromRows must accept at least one row [recovered]
> > panic: BUG: Inmemory.InitFromRows must accept at least one row
> > 
> > goroutine 31839 [running]:
> > testing.tRunner.func1.2({0x6f7a60, 0xc003b2e4b0})
> > /usr/lib/go-1.20/src/testing/testing.go:1526 +0x24e
> > testing.tRunner.func1()
> > /usr/lib/go-1.20/src/testing/testing.go:1529 +0x39f
> > panic({0x6f7a60, 0xc003b2e4b0})
> > /usr/lib/go-1.20/src/runtime/panic.go:884 +0x213
> > github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.logMessage({0x742b44, 
> > 0x5}, {0xc00507f300, 0x37}, 0x0?)
> > 
> > /<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:269
> >  +0x965
> > github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.logLevelSkipframes(0x1,
> >  {0x742b44, 0x5}, {0x75c548?, 0xc00503fcc8?}, {0x0?, 0x90?, 0x72aba0?})
> > 
> > /<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:137
> >  +0x1a5
> > github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.logLevel(...)
> > 
> > /<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:129
> > github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.Panicf(...)
> > 
> > /<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:125
> > github.com/VictoriaMetrics/VictoriaMetrics/lib/storage.(*inmemoryPart).InitFromRows(0xc7ccf0,
> >  {0x0, 0x0, 0x0})
> > 
> > /<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/storage/inmemory_part.go:37
> >  +0x5e
> > github.com/VictoriaMetrics/VictoriaMetrics/lib/storage.newTestBlockStreamReader(0xca6150?,
> >  {0x0, 0x0, 0x0})
> > 
> > /<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/storage/block_stream_reader_test.go:151
> >  +0x48
> > github.com/VictoriaMetrics/VictoriaMetrics/lib/storage.TestMergeForciblyStop(0xc004101380)
> > 
> > /<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/storage/merge_test.go:358
> >  +0xce
> > testing.tRunner(0xc004101380, 0x7e12e8)
> > /usr/lib/go-1.20/src/testing/testing.go:1576 +0x10b
> > created by testing.(*T).Run
> > /usr/lib/go-1.20/src/testing/testing.go:1629 +0x3ea
> > FAILgithub.com/VictoriaMetrics/VictoriaMetrics/lib/storage  10.215s

I cannot reproduce this one on my sid chroot, but another error
completely different in TestMarshalUnmarshalInt64Array.

Thanks,
Guillem

Bug#1043118: victoriametrics: FTBFS: test failure

2023-08-08 Thread Denys Holius 

Hello folks!

The latest version of VictoriaMetrics v1.92.0 
 should be marked 
as LTS but it has some backwards-incompatible changes. Meanwhile v1.87.6 
 was marked as 
LTS and maybe this version contains bugfix.


1. Could you please point me to a documentation which I can use to
   reproduce this bug?
2. Maybe we can make a bugfix or patch and send a PR to
   https://salsa.debian.org/go-team/packages/victoriametrics ?

On 06.08.23 12:16, Sebastian Ramacher wrote:

Source: victoriametrics
Version: 1.79.5+ds1-2
Severity: serious
Tags: ftbfs sid trixie
Justification: fails to build from source (but built successfully in the past)
X-Debbugs-Cc:sramac...@debian.org

https://buildd.debian.org/status/fetch.php?pkg=victoriametrics&arch=amd64&ver=1.79.5%2Bds1-2%2Bb6&stamp=1691274080&raw=0

=== RUN   TestMergeForciblyStop
2023-08-05T22:21:13.980Zpanic   
VictoriaMetrics/lib/storage/inmemory_part.go:37 BUG: Inmemory.InitFromRows must 
accept at least one row
--- FAIL: TestMergeForciblyStop (0.00s)
panic: BUG: Inmemory.InitFromRows must accept at least one row [recovered]
panic: BUG: Inmemory.InitFromRows must accept at least one row

goroutine 31839 [running]:
testing.tRunner.func1.2({0x6f7a60, 0xc003b2e4b0})
/usr/lib/go-1.20/src/testing/testing.go:1526 +0x24e
testing.tRunner.func1()
/usr/lib/go-1.20/src/testing/testing.go:1529 +0x39f
panic({0x6f7a60, 0xc003b2e4b0})
/usr/lib/go-1.20/src/runtime/panic.go:884 +0x213
github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.logMessage({0x742b44, 
0x5}, {0xc00507f300, 0x37}, 0x0?)

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:269
 +0x965
github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.logLevelSkipframes(0x1, 
{0x742b44, 0x5}, {0x75c548?, 0xc00503fcc8?}, {0x0?, 0x90?, 0x72aba0?})

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:137
 +0x1a5
github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.logLevel(...)

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:129
github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.Panicf(...)

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:125
github.com/VictoriaMetrics/VictoriaMetrics/lib/storage.(*inmemoryPart).InitFromRows(0xc7ccf0,
 {0x0, 0x0, 0x0})

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/storage/inmemory_part.go:37
 +0x5e
github.com/VictoriaMetrics/VictoriaMetrics/lib/storage.newTestBlockStreamReader(0xca6150?,
 {0x0, 0x0, 0x0})

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/storage/block_stream_reader_test.go:151
 +0x48
github.com/VictoriaMetrics/VictoriaMetrics/lib/storage.TestMergeForciblyStop(0xc004101380)

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/storage/merge_test.go:358
 +0xce
testing.tRunner(0xc004101380, 0x7e12e8)
/usr/lib/go-1.20/src/testing/testing.go:1576 +0x10b
created by testing.(*T).Run
/usr/lib/go-1.20/src/testing/testing.go:1629 +0x3ea
FAILgithub.com/VictoriaMetrics/VictoriaMetrics/lib/storage  10.215s

Cheers

Bug#1043118: victoriametrics: FTBFS: test failure

2023-08-06 Thread Sebastian Ramacher 
Source: victoriametrics
Version: 1.79.5+ds1-2
Severity: serious
Tags: ftbfs sid trixie
Justification: fails to build from source (but built successfully in the past)
X-Debbugs-Cc: sramac...@debian.org

https://buildd.debian.org/status/fetch.php?pkg=victoriametrics&arch=amd64&ver=1.79.5%2Bds1-2%2Bb6&stamp=1691274080&raw=0

=== RUN   TestMergeForciblyStop
2023-08-05T22:21:13.980Zpanic   
VictoriaMetrics/lib/storage/inmemory_part.go:37 BUG: Inmemory.InitFromRows must 
accept at least one row
--- FAIL: TestMergeForciblyStop (0.00s)
panic: BUG: Inmemory.InitFromRows must accept at least one row [recovered]
panic: BUG: Inmemory.InitFromRows must accept at least one row

goroutine 31839 [running]:
testing.tRunner.func1.2({0x6f7a60, 0xc003b2e4b0})
/usr/lib/go-1.20/src/testing/testing.go:1526 +0x24e
testing.tRunner.func1()
/usr/lib/go-1.20/src/testing/testing.go:1529 +0x39f
panic({0x6f7a60, 0xc003b2e4b0})
/usr/lib/go-1.20/src/runtime/panic.go:884 +0x213
github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.logMessage({0x742b44, 
0x5}, {0xc00507f300, 0x37}, 0x0?)

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:269
 +0x965
github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.logLevelSkipframes(0x1, 
{0x742b44, 0x5}, {0x75c548?, 0xc00503fcc8?}, {0x0?, 0x90?, 0x72aba0?})

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:137
 +0x1a5
github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.logLevel(...)

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:129
github.com/VictoriaMetrics/VictoriaMetrics/lib/logger.Panicf(...)

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/logger/logger.go:125
github.com/VictoriaMetrics/VictoriaMetrics/lib/storage.(*inmemoryPart).InitFromRows(0xc7ccf0,
 {0x0, 0x0, 0x0})

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/storage/inmemory_part.go:37
 +0x5e
github.com/VictoriaMetrics/VictoriaMetrics/lib/storage.newTestBlockStreamReader(0xca6150?,
 {0x0, 0x0, 0x0})

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/storage/block_stream_reader_test.go:151
 +0x48
github.com/VictoriaMetrics/VictoriaMetrics/lib/storage.TestMergeForciblyStop(0xc004101380)

/<>/_build/src/github.com/VictoriaMetrics/VictoriaMetrics/lib/storage/merge_test.go:358
 +0xce
testing.tRunner(0xc004101380, 0x7e12e8)
/usr/lib/go-1.20/src/testing/testing.go:1576 +0x10b
created by testing.(*T).Run
/usr/lib/go-1.20/src/testing/testing.go:1629 +0x3ea
FAILgithub.com/VictoriaMetrics/VictoriaMetrics/lib/storage  10.215s

Cheers
-- 
Sebastian Ramacher