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

Adding initialization, build and test scripts. Modifying mysql host name on hibernate.cfg.xml. Adding healthcheck param on docker-compose. #4

Merged
merged 5 commits into from
Sep 2, 2018

Conversation

siriusgomes
Copy link
Collaborator

Adding initialization, build and test scripts. Modifying mysql host name on hibernate.cfg.xml. Adding healthcheck param on docker-compose.

Sírius Roberto da Costa Gomes added 2 commits August 31, 2018 16:09
…ame on hibernate.cfg.xml. Adding healthcheck param on docker-compose.
run.ps1 Outdated
@@ -0,0 +1,2 @@
echo "Booting docker-compose production file"
Copy link
Owner

Choose a reason for hiding this comment

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

Acho melhor deixar esses scripts em uma pasta separada. Uma pasta scripts talvez

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Consider it done as well!

build.ps1 Outdated
@@ -0,0 +1,15 @@
echo "Stopping all running docker containers"
Copy link
Owner

Choose a reason for hiding this comment

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

Acho melhor deixar esses scripts em uma pasta separada. Uma pasta scripts talvez

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Consider it done!

Copy link
Owner

@helton helton left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -25,6 +25,11 @@ services:
build: ../backend/
ports:
- 8090:8080
healthcheck:
Copy link
Owner

@helton helton Sep 1, 2018

Choose a reason for hiding this comment

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

O healthcheck tem que ser feito no mysql e o backend tem que ser iniciado apenas se condition: service_healthy
Vi que esse o depends_on com condition só funciona nas versões 2.x do docker-compose por uma incompatibilidade com o docker swarm.

Acho melhor usar a versão 2.x então, igual esse exemplo do StackOverflow

Vou alterar aqui e já mando um commit nessa branch pra aparecer nesse PR.

@siriusgomes
Copy link
Collaborator Author

Conforme comentado, movi os scripts para uma pasta separada, aproveitei para criar scripts shell.

@siriusgomes siriusgomes merged commit 3afe970 into master Sep 2, 2018
@helton helton self-assigned this Sep 3, 2018
@helton helton assigned siriusgomes and helton and unassigned helton Sep 3, 2018
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