-
Notifications
You must be signed in to change notification settings - Fork 37
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
[#4] Create Docker for Development #27
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.
@mappuji very very awesome
can you change the file names to accommodate future file structure
docker
├── db-init
│ ├── Dockerfile
│ └── rpi-Dockerfile
├── planet
│ ├── Dockerfile
│ └── rpi-Dockerfile
├── planet-dev.yml
├── planet.yml
├── rpi-planet-dev.yml
└── rpi-planet.yml
docker-dev/README.md
Outdated
## Description | ||
This docker compose can be use for your development environment and very handy, you can spawn the development environment in a matter of seconds and start your development. Your code changes in host folder are automatically reflected to docker and ready to test in your browser. | ||
|
||
## How to use |
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.
Add how to use it
tells about docker logs
docker-dev/docker-compose.yml
Outdated
couchdb: | ||
expose: | ||
- 5984 | ||
image: klaemo/couchdb |
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.
Better to use official docker image I think
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.
thanks
docker/planet/docker-entrypoint.sh
Outdated
cd - | ||
# End add CORS to CouchDB | ||
|
||
# Add initial Couch databases here |
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.
remove trailing and following lines ;) thanks @mappuji
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.
or just remove it ...
docker/db-init/docker-entrypoint.sh
Outdated
git clone https://github.com/pouchdb/add-cors-to-couchdb.git | ||
cd add-cors-to-couchdb | ||
npm install | ||
node bin.js http:https://couchdb:5984 |
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.
move to after wait for couchdb
docker/db-init/docker-entrypoint.sh
Outdated
echo "..." | ||
sleep 5 | ||
WAIT_TIME=$(expr $WAIT_TIME + 5) | ||
done |
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.
check if couchdb is already initialized
docker/planet/Dockerfile
Outdated
VOLUME ["/$APP"] | ||
WORKDIR /$APP | ||
|
||
ADD ./docker-entrypoint.sh docker-entrypoint.sh |
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.
;)
@dogi and @paulbert we need to arrange a demo session for this one.
Basically in this PR I want to increase the development environment for faster development using docker and docker-compose. Checkout my PR and see readme for testing.
Solved #4