Skip to content

Dockerize ncharts - #19

Open
erikj wants to merge 48 commits into
developfrom
feature-15-docker-nginx-unicorn
Open

Dockerize ncharts#19
erikj wants to merge 48 commits into
developfrom
feature-15-docker-nginx-unicorn

Conversation

@erikj

@erikj erikj commented Oct 18, 2016

Copy link
Copy Markdown
Contributor

Pull request for review and discussion:

  • dockerizes ncharts by
    • adding configuration to utilize official Python image and support required services via Docker Compose
    • moving hard-coded production paths to environment variables, namespaced w/ NCHARTS_ prefix
  • switch from Apache + mod_wsgi to Nginx + gunicorn, for lighter and easier to debug environment
  • config: docker/
  • documentation: docs/DOCKER.md, docker/README.md
  • demo: http://datavis-dev.eol.ucar.edu:8000/

Resolves #15

… for running ncharts Django application w/ gunicorn; #15
…r than just unix socket, via s/VAR_RUN_DIR/MEMCACHED_LOCATION/; #15
@erikj

erikj commented Oct 18, 2016

Copy link
Copy Markdown
Contributor Author

This isn't the most atomic pull request, but some of the big advantages that it provides:

  • Moves the Python execution environment out of package management, so we don't have to build and maintain our own Python 3 packages. Staying up to date on security fixes will be much easier.
  • Decouples the web server and the execution environment dependencies by removing mod_wsgi.
  • Makes spinning up a new environment much easier, faster, consistent and reliable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nginx?

2 participants