Skip to content
This repository has been archived by the owner on Feb 1, 2019. It is now read-only.

Vagrant and Docker refactor #813

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Vagrant and Docker refactor #813

wants to merge 4 commits into from

Conversation

robdyke
Copy link

@robdyke robdyke commented Jan 1, 2019

  • Mount project folder to /home/vagrant/good-thinking
  • Run script unpriviledged
  • Use sudo -H to install node and python packages

@robdyke
Refactored inline script
decdf4b 
- Mount project folder to /home/vagrant/good-thinking
- Run script unpriviledged
- Use sudo -H to install node and python packages
@robdyke
Copy link
Author

robdyke commented Jan 1, 2019

I still want to add a multi machine setup for Vagrant, separating db from app, to speed up the provisioning.

robdyke added 3 commits January 1, 2019 20:31
@robdyke
Vagrant multimachine setup
d10a5e6 
  - Do not update guest additions by default

Web machine
  - vagrant up web
  - Install nodejs-dev rather than build-essential
  - Removed sudo

DB machine
  - vagrant up db
  - IP 192.168.33.21

Dockerised machine
  - vagrant up docker
@robdyke
Docker container
463c648 
  - Support multistage builds
  - Reduced layer sizes
  - Add LABELS
@robdyke
Additional comments to Vagrantfile
917df55
@robdyke robdyke changed the title Refactored inline script Vagrant and Docker refactor Jan 1, 2019
@robdyke
Copy link
Author

robdyke commented Jan 1, 2019

Added Vagrant multi-machine environment
Added Vagrant Docker definition
Added multistage build to Dockerfile

@robdyke
Copy link
Author

robdyke commented Jan 1, 2019

Python 3.6 is updated to latest in Vagrantfile and Dockerfile which addresses #797

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@robdyke