-
Notifications
You must be signed in to change notification settings - Fork 10
DevOps Wishlist #176
Comments
From our discussion, the following seem like the most value add: TODO:
Deemed Unnecessary:
|
TODO from 2021-05-07 Stream:
Issue was that the DB was not getting properly seeded/migrated in the docker-compose setup. Modified bootstrap.sh to use with docker compose. |
those commands inside bootstrap.sh, are they being executed inside docker containers ? |
Previously they were not being executed inside the container. I updated the CMD in the server Dockerfile here: https://github.com/sidpalas/leaguedex/blob/8fa930a8ec8963e34ee4ab9b6111e6dba18161fb/server/Dockerfile#L17 For the non-development version of the image we will probably want to extract it out such that the seeding/migration of the DB is separate from the application, but that was the fastest way to get it working for the dev environment! |
you are only migrating/seeding the database schema here, am I right ? |
The text was updated successfully, but these errors were encountered: