Fix setup for development with docker #523
Conversation
wvengen
changed the title
wvengen
force-pushed
the
feature/docker-dev
branch
from
d01901c
to
85088a3
Compare
wvengen
force-pushed
the
feature/docker-dev
branch
from
85088a3
to
bc2da15
Compare
|
This uid hack may still be useful, keeping that as a later improvement. |
wvengen
changed the title
docker-compose-dev.yml
Outdated
| version: '2' | ||
| services: | ||
|
|
||
| web: |
There was a problem hiding this comment.
I'd like to align this with our foodcoops.net setup
docker-compose-dev.yml
Outdated
| services: | ||
|
|
||
| web: | ||
| extends: worker |
There was a problem hiding this comment.
why does web extend worker and not the other way around?
There was a problem hiding this comment.
Because ports would be added creating a conflict. Need to rework setup to avoid this issue, but I'd like to resolve the current non-working docker-based dev setup asap.
| - QUEUE=foodsoft_notifier | ||
| - TEST_DATABASE_URL=mysql2://root:secret@mariadb/test?encoding=utf8 | ||
|
|
||
| mailcatcher: |
There was a problem hiding this comment.
i see not configuration sending the mails to this container
There was a problem hiding this comment.
Took same approach as before, where current configuration detects docker environment variables and connects to mailcatcher. Later we can change this to a proper smtp url.
|
Ok to merge? I'll do so today without response. |
This completes #449 and #491. Reorganizes development and production docs somewhat, and brings back the development setup using docker. It could be improved, but I think this is already a small step forward.
After this is merged, the wiki page Deployment notes may need updated links.