Skip to content

Commit

Permalink
updated readme
Browse files Browse the repository at this point in the history
  • Loading branch information
Tomasz Szawara authored and Tomasz Szawara committed Sep 10, 2017
1 parent a762f4c commit 0a284ec
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions backend/db/DB-README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,15 @@ Docker container to provide database for development & production.
# Pre-requisites

## In production:
Make sure you:
* [understand user-defined networks](https://docs.docker.com/engine/userguide/networking/#user-defined-networks)
Make sure you:
* [understand user-defined networks](https://docs.docker.com/engine/userguide/networking/#user-defined-networks)
* have your network created: `docker network create network-1`
* have other container who use this database join this network. Read: [connecting containers](https://docs.docker.com/engine/userguide/networking/work-with-networks/#connect-containers)
* note: `link` option may be used here (its not the same as legacy`link` for default bridge network).

# Run
Build Image
* `docker build -t words-db .`
* `docker build . -t words-db .`

Stop previous container if running
* `docker stop words-api-box`
Expand All @@ -30,11 +30,10 @@ Useful Queries:
* none (queries are simple, check backend code for examples)


#Road-map:
#Road-map:
* Consider whether KNEX migration belongs in DB or API or separate
** when I deploy db I want db to be up to date
** when I deploy API ... I probably just want to deploy API (but I might also be deploying new features which require db schema update)
** API will be deployed way more often then schema (but DB container itself might never be re-deployed)
** running knex when there's no migration shouldn't take more then a split second
** I don't deploy API via docker on local (can I deploy an already migrated DB container?)

0 comments on commit 0a284ec

Please sign in to comment.