-
Notifications
You must be signed in to change notification settings - Fork 61
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
Remove centos 7 docker image #1791
Conversation
@@ -93,7 +92,7 @@ jobs: | |||
retention-days: 1 | |||
|
|||
build-jvm-linux-native-lib: | |||
runs-on: ubuntu-latest | |||
runs-on: ubuntu-22.04 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What version of libc
it has? the goal with old centos:7
is to build it on the smallest libc Core supports (glibc 2.17). So maybe we can still use/install the old libc via ubuntu
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - Only some minor comments.
Centos 7 has been deprecated as of 30th June 2024. Our CI job
build-jvm-linux-native-lib
used it in a docker image.This PR removes the docker image and integrates this build job in GHA using a ubuntu-22.04 image.
This is the same image as used by core: https://github.com/realm/ci/blob/master/realm/docker/build-ubuntu/Dockerfile
This PR includes the changes from #1782:
Some schema synchronization issues caused the app not being properly initialized. We have a workaround that addresses them, but unfortunately there are still some race conditions.
Tests now define the schema in the server via the admin API instead of relying on the development mode to do the synchronization. It allows to first set up the server, enable sync, and wait for sync to be enabled. Once enabled we wait to be ready to accept new clients.