Skip to content
This repository has been archived by the owner on Jan 12, 2025. It is now read-only.

Docker compatibility #72

Merged
merged 8 commits into from
Aug 15, 2024
Merged

Docker compatibility #72

merged 8 commits into from
Aug 15, 2024

Conversation

AlexandreVassard
Copy link

@AlexandreVassard AlexandreVassard commented Jul 30, 2024

Hey,

I wanted to make Kepler compatible with Docker.

So i made a Dockerfile (based on alpine to have the smallest image possible).

The entrypoint will, at first init, run Kepler to get the default server.ini

After that, it will take all values from .env and put them in server.ini

WARN : mysql.username is MYSQL_USER because it's also used in MariaDB.

The Dockerfile does the same thing that is done in Github actions (CI) for a release.

If you have any questions or need help to deploy Docker image with CI, please ask.

Windows could not build because of CRLF/LF files configuration.

So Dockerfile converts files to LF.
We don't need .bat file as Dockerfile uses alpine (linux)
For those who uses vscode
Docker will keep in cache the build until source is updated
Also refactor some code in Dockerfile
Copy link

@AqueleHaru AqueleHaru left a comment

Choose a reason for hiding this comment

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

please!

@emansom
Copy link

emansom commented Aug 13, 2024

Re-post of what I posted in Quackster/Havana#50, as the same applies here:

Some feedback:

  • Modify the code to read from environment variables instead of relying on server.ini modification. (supporting both, prioritizing env above ini)
  • Build the container images using jib instead. This will save on build time and total image size.

https://cloud.google.com/java/getting-started/jib
https://www.youtube.com/watch?v=H6gR_Cv4yWI

https://github.com/emansom/Kepler/blob/master/Kepler-Server/build.gradle#L18

@Quackster Quackster merged commit e5df356 into Quackster:master Aug 15, 2024
AlexandreVassard added a commit to AlexandreVassard/Kepler that referenced this pull request Aug 27, 2024
Merge pull request Quackster#72 from AlexandreVassard/master
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.

5 participants