chime icon indicating copy to clipboard operation
chime copied to clipboard
verified publisher icon pennsignals

Getting ready to deploy on k8s

Open sam-writer opened this issue 6 years ago • 1 comments

In preparation to deploy on k8s (https://github.com/CodeForPhilly/civic-cluster/issues/6), this PR:

  • Changed: moves configs from setup.sh into TOML files in ./.streamlit/ to centralize them

  • Changed: Dockerfile base image is slim-buster. This one line cuts the image size in half, from 1.37GB to 630MB

  • Changed: Dockerfile no longer reference script.sh, configs moved to .streamlit dir

  • Changed: Dockerfile CMD command doesn't specific server.port, which is already specified in ./streamlit/config.toml

  • Changed: Dockerfile add -q flag to pip install to make build logs shorter

  • Added: a .dockerignore to avoid copying unneeded files into the docker image

sam-writer avatar Mar 16 '20 18:03 sam-writer

FYI this DOES NOT block deploying on k8s, all these changes are cleanup/performance improvements, e.g. nice-to-haves

sam-writer avatar Mar 16 '20 18:03 sam-writer