Control: severity -1 import

On Sun, Dec 13, 2020 at 2:15 AM Andreas Henriksson <andr...@fatal.se> wrote:
>
> Hello!
>
> On Wed, Dec 09, 2020 at 10:03:39AM +0100, Lucas Nussbaum wrote:
> > Source: golang-github-coreos-bbolt
> > Version: 1.3.5-1
> > Severity: serious
> > Justification: FTBFS on ppc64el
> > Tags: bullseye sid ftbfs
> > Usertags: ftbfs-20201209 ftbfs-bullseye ftbfs-ppc64el
> >
> > Hi,
> >
> > During a rebuild of all packages in sid, your package failed to build
> > on ppc64el. At the same time, it did not fail on amd64.
> [...]
> > > === RUN   TestBucket_Stats
> > >     bucket_test.go:1222: unexpected BranchPageN: 0
> > > --- FAIL: TestBucket_Stats (9.56s)
> [...]
>
> Apparently bbolt is a fork of boltdb, so please see my comments
> on the equivalent bug report againt boltdb:
> https://bugs.debian.org/976907
>
>
> Relevant part:
>  It seems the test-suite makes assumptions related to calculations that
>  involve os.Getpagesize() (which gives 4096 on amd64 and 65536 on
>  ppc64el, which is 16 times larger).
>  Changing the 500 number to 8000 (16 times larger) in
>  TestBucket_Stats(...) (in bucket_test.go:1143) gives the expected
>  BranchPageN == 1 .... (however after that it then says
>  "unexpected LeafPageN: 6" with this modification).
>
>  In my opinion it's pretty clear that these are test-suite only
>  issues and not issues in the actual product.
>
>
> I would thus personally just disable the test-suite on !amd64 if no
> porter is interested in fixing the testsuite (unless we agree this
> issue simply can be downgraded to non-RC).

I prefer to downgrade the severity. Just the test, not the library
itself is broken.

-- 
Shengjing Zhu

Reply via email to