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

Problems with Rocksdb #1632

Open
pSchlarb opened this issue Feb 24, 2023 · 5 comments
Open

Problems with Rocksdb #1632

pSchlarb opened this issue Feb 24, 2023 · 5 comments
Labels
CI/CD CI/CD related issue help wanted

Comments

@pSchlarb
Copy link
Contributor

pSchlarb commented Feb 24, 2023

Plenums 3rd parties dependencies use a build of rocksdb from evernym. (https://github.com/evernym/rocksdb)

The built of rocksdb from evernym is not declaring its dependencies such as:
libgflags-dev
libsnappy-dev
zlib1g-dev
libbz2-dev
liblz4-dev
libgflags-dev

References:

function build_rocksdb_deb {
VERSION=$1
VERSION_TAG="rocksdb-$VERSION"
git clone https://github.com/evernym/rocksdb.git /tmp/rocksdb
cd /tmp/rocksdb
git checkout $VERSION_TAG
sed -i 's/-m [email protected]/-m "Hyperledger <[email protected]>"/g' \
./build_tools/make_package.sh
PORTABLE=1 EXTRA_CFLAGS="-fPIC" EXTRA_CXXFLAGS="-fPIC" ./build_tools/make_package.sh $VERSION
# Install it in the system as it is needed by python-rocksdb.
make install
cd -
cp /tmp/rocksdb/package/rocksdb_${VERSION}_amd64.deb $OUTPUT_PATH
rm -rf /tmp/rocksdb
}

@WadeBarnes
Copy link
Member

WadeBarnes commented Feb 24, 2023

When upgrading, investigate using an official package. One that already defines the dependencies properly, hopefully.

@WadeBarnes
Copy link
Member

The other issue we have to the Python wrapper being used for rocksdb is out of date, and there seems to be a fracturing of replacement wrappers for it.

@pSchlarb
Copy link
Contributor Author

pSchlarb commented Mar 9, 2023

@WadeBarnes should we close this issue, since we have a working workaround for the moment?
Or should it stay open as a reminder for the rocksdb upgrades?

@WadeBarnes
Copy link
Member

It should stay open. Thanks @pSchlarb

@manolaz
Copy link

manolaz commented Jul 19, 2023

alright got it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD CI/CD related issue help wanted
Projects
None yet
Development

No branches or pull requests

3 participants