-
Notifications
You must be signed in to change notification settings - Fork 12
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
Chore/fix bugs #27
base: master
Are you sure you want to change the base?
Chore/fix bugs #27
Conversation
Update postgres version 11 to 17
Fix order of parameters
Update network and remove services that were causing errors
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.
I need to think more about maintain the postgres yaml file at this repository.
Maybe would be good to have a dedicated repository with more instructions about PostgreSQL.
postgres: | ||
external: true | ||
name: postgres |
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.
This network is necessary to be possible connect the service app at this network.
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.
Actually service app uses only internal network.
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 do you think about create a new repository specific to PostgreSQL and to put all documentation about PostgreSQL at the newest repository?
Co-authored-by: Vitor Mattos <[email protected]>
Update version of postgres and remove services that were not used and were causing errors