Skip to content
This repository has been archived by the owner on May 25, 2021. It is now read-only.

YACS on Docker! #135

Merged
merged 84 commits into from
Jan 6, 2017
Merged

YACS on Docker! #135

merged 84 commits into from
Jan 6, 2017

Conversation

Bad-Science
Copy link
Member

@Bad-Science Bad-Science commented Sep 16, 2016

Resolves #127 - meets specified acceptance criteria.

Docker images:

  • rails: The actual YACS application
  • nginx: based off of official nginx image with an added config file
  • postgres: official postgres image (9.5)
  • redis: official redis image

Docker services:

  • web: rails container that runs puma
  • worker: rails container that runs crono (jobs worker)
  • nginx: container running nginx build for serving static files. uses a reverse proxy to forward requests for non-static files to web
  • redis: container running redis image
  • postgres: container running postgres image

Still to-do:

  • Add SSL config to new nginx and puma configs & disable in development

The services are set up with docker-compose (see docker-compose.yml). The app source on the host is set up as a volume and shared between web, worker and nginx services.

The local setup process is defined in https://github.com/YACS-RCOS/yacs/wiki/Local-Setup and is much easier on mac os and windows than it was prior.

- script/yacs-build
- script/yacs-startd && docker ps
- sleep 10
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Bad-Science
Copy link
Member Author

Bad-Science commented Dec 6, 2016

SSL config done with #165

However there not appears to be a bug, nginx does not appear to be passing assets requests to rails in development.

edit: fixed with #188

@Bad-Science
Copy link
Member Author

This is done and has been in production for almost a week with no complaints. Merging and releasing.

@Bad-Science Bad-Science merged commit 7c3b5d4 into YACS-RCOS:staging Jan 6, 2017
Bad-Science added a commit that referenced this pull request Aug 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority:high This issue has a high priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants