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(docker): set up a local shared postgres for development #1272
Conversation
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.
Nice! Just one nit.
docker-compose.dev.yml
Outdated
@@ -30,6 +30,22 @@ services: | |||
resource-recorder: | |||
ports: | |||
- 8007:8000 | |||
depends_on: | |||
- auth |
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.
Nit: The .dev.yml file will be combined with the docker-compose.yml into docker-compose.rendered.yml for local development, so we don't need to specify the auth dependency here. I see we did the same thing for logger in this file.
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.
Description of change
This PR updates Makefile/
docker-compose
file for creating a local/shared Postgres database called "control DB" for development.I also realized we already have a shared Postgres at
/dev/tcp/postgres/5432
which begs the question do we still need this control DB?How has this been tested? (if applicable)
N/a