-
Notifications
You must be signed in to change notification settings - Fork 111
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
The wallet test fails with valgrind #82
Comments
The test still fails. rustc -V
rustc 1.64.0 (a55dd71d5 2022-09-19) |
May be related to the test coredumps in Alpine Linux v3.16 with rust 1.64. exe=target/release/deps/darkfi-96fc625923ef093a
tests='wallet::walletdb::tests::test_walletdb zk::circuit::burn_contract::tests::burn_circuit_assert zk::circuit::mint_contract::tests::mint_circuit_assert zk::gadget::arithmetic::tests::arithmetic_circuit_assert zk::gadget::less_than::tests::less_than_64 zk::gadget::native_range_check::tests::native_range_check_128 zk::gadget::native_range_check::tests::native_range_check_253 zk::gadget::native_range_check::tests::native_range_check_64'
for test in $tests; do $exe --test --exact $test;done Expected: ok Actual: Segmentation fault (core dumped) |
I was looking at this, and it seems to be internally related to SQLite. We have plans in the future to migrate to our own internal Db. This is just used during the bootstrapping phase and it's likely not to be a major show stopper to getting darkfi online. But yeah we will slowly deprecate this portion of the code, and develop a new replacement module for WalletDb. |
The latest sources fail to test properly in a container build, even tests pass in normal build. I suspect the problem is again around libsqlite3 / openssl-sys. Not found if |
The problem manifested first in a docker build using Alpine Linux image.
Reproduced on hardware - notebook with Fedora 36 and with the latest commit d0ae3f3 in a Fedora 36 in a VM, 4 cores, 17 GB RAM:
The text was updated successfully, but these errors were encountered: