Skip to content
This repository was archived by the owner on Jun 9, 2020. It is now read-only.

Fix problem with rocksdb on alpine based image #22

Merged
merged 1 commit into from
Sep 8, 2017
Merged

Fix problem with rocksdb on alpine based image #22

merged 1 commit into from
Sep 8, 2017

Conversation

iemejia
Copy link
Member

@iemejia iemejia commented Aug 1, 2017

Alpine has a libc6-compat library that makes rocksdb work on it

@iemejia
Copy link
Member Author

iemejia commented Aug 1, 2017

@patricklucas also worth taking a look, and also if your tests go well worth releasing finally the alpine version image.

@patricklucas
Copy link
Member

Hmm, as noted in #14 I tried the libc6-compat library but still got a (different) error. That doesn't happen now when you use the same 'steps to reproduce'?

@iemejia
Copy link
Member Author

iemejia commented Aug 1, 2017

I tested exactly the same steps you mention with no exception so far, maybe it is also because the image is now based on a more recent version of alpine I think.

Alpine has a libc6-compat library that makes rocksdb work on it
@iemejia
Copy link
Member Author

iemejia commented Sep 7, 2017

@patricklucas this has been waiting for some time to be merged, PTAL.

@@ -18,8 +18,8 @@

FROM openjdk:8-jre-alpine

# Install dependencies, Bash, and su-exec for easy step-down from root
RUN apk add --no-cache bash snappy 'su-exec>=0.2'
# Install dependencies, bash and su-exec for easy step-down from root
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change seems unnecessary, but not a blocker to merging I suppose.

@patricklucas patricklucas merged commit 01fdc88 into docker-flink:master Sep 8, 2017
@patricklucas
Copy link
Member

Haven't had a chance to test it, but I'll take your word for it.

@patricklucas
Copy link
Member

I caved and decided to properly verify this—indeed, if you use an older version of alpine even with libc6-compat, the crash occurs, whereas in the version of alpine that the current version of openjdk:8-jre-alpine builds on, it does not.

I'll push the new Dockerfiles upstream shortly.

@honestAnt
Copy link

honestAnt commented Dec 4, 2019

add it in dockerfile;
apk update &&
apk add --no-cache libc6-compat &&
ln -s /lib/libc.musl-x86_64.so.1 /lib/ld-linux-x86-64.so.2

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants