Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TestBadger fails on 32-bit / i386 linux #25

Open
pymnh opened this issue Sep 9, 2022 · 1 comment
Open

TestBadger fails on 32-bit / i386 linux #25

pymnh opened this issue Sep 9, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@pymnh
Copy link

pymnh commented Sep 9, 2022

Subject of the issue

TestBadger fails when running reproducibility testing on 32-bit / i386 debian

Your environment

  • OS: Debian
  • Version: unstable
  • Architecture: i386

Additional context

=== RUN   TestBadger
badger 2022/08/01 05:59:21 INFO: All 0 tables opened in 0s
    nosql_test.go:322: error 'error opening Badger database: During db.vlog.open: Error while creating log file in valueLog.open: Mmap value log file. Path=./tmp/badgerdb/000000.vlog. Error=cannot allocate memory' not expected
--- FAIL: TestBadger (0.06s)

For full logs see here: https://tests.reproducible-builds.org/debian/rb-pkg/bookworm/i386/golang-github-smallstep-nosql.html

@pymnh pymnh added bug Something isn't working needs triage Waiting for discussion / prioritization by team labels Sep 9, 2022
@dopey
Copy link
Contributor

dopey commented Sep 21, 2022

Hey @pymnh thanks for opening the issue!

Truth be told I haven't dug into the Badger integration in a looong time. However, I recall that specific architectures require Badger to be run with configuration flags that restrict the memory, disk, addressable bit space.

I think the error you're seeing is a known Badger error (dgraph-io/badger#246) -- it's Badger yelling that it doesn't have enough room to run. Since we generally test in 64 bit environments, we haven't optimized the tests to run in 32 bit environments. If you'd like to post a PR with the changes necessary to run the tests against a 32 bit env, we'll happily merge. I imagine it would just be adding some options here -

db, err := New("badger", path, WithValueDir(path))
.

Cheers 🍻

@dopey dopey removed the needs triage Waiting for discussion / prioritization by team label Jan 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants