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.

Sign up for GitHub

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

Use Gunicorn in the Docker container #8676

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Use Gunicorn in the Docker container #8676

wants to merge 11 commits into from

Commits on Dec 11, 2024

  1. Proof-of-concept with Gunicorn 

    This serves the Django application and static assets with Gunicorn from the "prod" Docker target container.
    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    da51f7e View commit details
    Browse the repository at this point in the history

  2. Add an Apache container 

    Instead of running Apache in our production container, run a separate Apache container that `ProxyPass`es to our Django/Gunicorn container.
    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    92d2ff4 View commit details
    Browse the repository at this point in the history

  3. Copy files separately for caching purposes

    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    ebc8786 View commit details
    Browse the repository at this point in the history

  4. Fix dev container by using node-builder container

    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    dc0587d View commit details
    Browse the repository at this point in the history

  5. Use DEPLOY_ENVIRONMENT for container-specific config 

    This changes moves our Gunicorn dependency to a separate requirements file, and moves container-specific Django configuration to a `DEPLOY_ENVIRONMENT==container` check, and adds a `DEPLOY_ENVIRONMENT` environment variable to the container.
    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    5c53835 View commit details
    Browse the repository at this point in the history

  6. Remove the generic, unused wsgi.conf

    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    ad76490 View commit details
    Browse the repository at this point in the history

  7. Create default Apache user/group config 

    For deployment to servers, we do not set APACHE_USER/APACHE_GROUP. This change sets their default to those expected values, to be overridden only if the variables are set in the environment.
    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    909582d View commit details
    Browse the repository at this point in the history

  8. Limit Apache proxying to specific env config 

    This change limits when Apache will proxy pass to the application to when the CFGOV_APPLICATION_HOST env var is set.
    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    88e8894 View commit details
    Browse the repository at this point in the history

  9. Add a non-root user to the container

    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    386f8bc View commit details
    Browse the repository at this point in the history

  10. Update our running in docker docs

    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    99a0bc8 View commit details
    Browse the repository at this point in the history

  11. Consolidate settings to avoid setting DEPLOY_ENVIRONMENT

    @willbarton
    willbarton committed Dec 11, 2024
    Configuration menu
    Copy the full SHA
    892b8cf View commit details
    Browse the repository at this point in the history