Skip to content
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 #233

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Docker #233

wants to merge 1 commit into from

Conversation

ekristen
Copy link

@ekristen ekristen commented Nov 3, 2020

This PR makes building an official docker image possible. I would still consider this a work in progress and needs further testing, however I wanted to open it up to gauge interest in actually merging it.

  1. Restructures the demo to just use the official docker image (once it's made)
  2. Restructures the contents of demo into app/ to build an actual django app inside the docker image.
  3. Introduces a multi-stage docker image that will assemble all the pieces required to run response.
  4. Ensures that all settings can be set via environment variables.

How To Test

  1. clone my fork
  2. docker build -t response .
  3. cp demo/env.example demo/.env and modify with settings
  4. cd demo && docker-compose up -d

Docker Image Setup

The django app is simply installed to /app and response is installed as an actual python module in Stage 1 of the docker build. In the final stage all install python modules are pulled down and made available in the final image, keeping the final image as small as possible.

The docker image utilizes a entrypoint script that was adapted from the demo script, and can be adapted further to support setting an ADMIN user/pass via environment variables. The script ensures the database is up, migrations are run, and then starts uwsgi up to run the django app.

TODOS

These are possible todos I'd like to do still if there's interest.

  • include helm chart
  • include kubernetes manifests/kustomize manifest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant