-
Notifications
You must be signed in to change notification settings - Fork 349
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
Docker-compose #121
Docker-compose #121
Conversation
@ohtake Looks like it just might be missing LaTeX distribution? Otherwise, this is a good base for adding the rest of the native dependencies, I will pull this down and run the tests. |
It installs TeX Live at line 32, but I have not tested the TeX feature. |
Did you try running this for dev environment which requires Postgres? AFAIK you need: |
Perhaps with this workflow we should look at dropping the separation between the two and just use MySQL in development and production. |
Sounds good to me :) That'd help with the emoji thingy as well. |
|
b79a3a6
to
a603caa
Compare
Two tests passed:
Shall I separte MySQL in this PR? |
@ohtake Looks good so far, let's just keep it in this PR, we can merge when ready 👍 |
I had a talk with @jakerenzella about reloading the API without restarting a Docker container. Similar approach may be applied to the web project but it requires modifications on Grunt tasks. |
- ffmpeg is available on newer versions of Ubuntu - ffmpeg replaces libav-tools - installs Ruby using rbenv because Ruby 2.3.1 is not provided by Ubuntu - wget is required by texlive-install.sh - libmysqlclient-dev is required to build gems
be2ffa9
to
cba9bac
Compare
Finally, managed to dockerize the api and web with live-reloading feature. Usageexport DF_DOCKER_MACHINE_IP=192.0.2.1 # Replace it for your own env
cd doubtfire-api
docker-compose build
docker-compose up -d
# wait a few seconds
docker-compose exec api bundle exec rake db:populate Open http://localhost:8000/ for Angular 1 project ('development' branch in web) or http://localhost:4200/ for Angular 6 project ('quality/decafinate' branch in web). Merging strategyBecause of dependencies and duplicated commits, it is better to merge PRs in the following order.
|
cba9bac
to
7dc8d41
Compare
This PR will: