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

Deployment with Docker Compose #292

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

wacken89
Copy link

@wacken89 wacken89 commented Sep 2, 2024

Hello,
in this PR I propose to run the application with docker-compose

Changes

  • Added docker-compose.yaml file to the root directory
  • Added PostgreSQL to docker-compose file. This change allows to run all application faster without configuring the database separately
  • Moved docker-entrypoint.sh to the root directory and added variable MOBILE_UTILITY_SERVER_DATABASE_MIGRATIONS. This helped merge two directories/phases (database and runtime). The difference was only in running the Java app at the end of the script
  • Added docker multi-stage build, now, the application can copy mobile-utility-server.war natively by docker instead of 3 shell commands
  • removed old files and directories

Run the application

docker compose up --build

P.S. Hope this PR will be helpful

Copy link
Member

@zcgandcomp zcgandcomp left a comment

Choose a reason for hiding this comment

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

Hello,

With our whole open source stack, we aim to consolidate how we work with build and docker.
Namely:

  • introduce a unified build of images,
  • introduce docker-compose for projects (for testing and development),
  • introduce separation of data migration and the application run.

This PR partially solves it, but as we want to do it the same way throughout the whole stack, we cannot use it as it is.

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

Successfully merging this pull request may close these issues.

2 participants