-
Notifications
You must be signed in to change notification settings - Fork 8
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
fix(docker): Use ubuntu and eclipse-temurin base image #14
fix(docker): Use ubuntu and eclipse-temurin base image #14
Conversation
Problem described in scylladb#13, missing libsnappy and linker loader errors requires us to change from alpine base to ubuntu base (default base image for `eclipse-temurin`). Some environmental variables for Linker loader have been added to point to the libraries installed by the package manager. Signed-off-by: Dusan Malusev <dusan.malusev@scylladb.com>
I can take it for a ride if needed, I'm in a middle of sessions building my own dockers and running some tests. |
If you have time, go for it. I will not merge this as there is some issue with building the image inside github actions, looks like credentials are not in order as seen in #15 PR |
@CodeLieutenant @fruch If I understand it correctly this fix has not landed yet (probably because of #15). Now almost all rolling upgrade tests for the latest 2024.1.8 RC are failing with the:
|
This fix has landed in |
@CodeLieutenant what is stopping us from updating SCT with the new image ? |
most likely nothing, it's just not well tested |
open a PR in SCT, and we'll see how to test it |
@CodeLieutenant can we plan adding this thing to SCT? It's still a blocker for testing rolling upgrades in 2014.1.8 |
Already Created PR 8477 |
Problem described in #13, missing libsnappy and linker loader errors requires us to change from alpine base to ubuntu base (default base image for
eclipse-temurin
).Some environmental variables for Linker loader have been added to point to the libraries installed by the package manager.