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 related support enhanced #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

mrunal4
Copy link

@mrunal4 mrunal4 commented Feb 5, 2020

Commit related short descriptions:

  • iniit_db.sql no more needed as database creation is handled in the docker-compose.yml and commented school_metric_data.sql content as it was just for testing as confirmed by Partha (Pardhu448)
  • version changed as it was prompting version mismatch warnings for urllib3 and psycopg2
  • To get command line arguments so that we can create users from scripts sliently (without user intervention/prompts). If arguments are missing it will prompt user for inputs
  • Changed DB_HOST and DB_PORT values
  • Check for - getting postgres up. Added : flask - init, migrate and upgrade along with admin user creation
  • Added support for env variables in Dockerfile, .env files and config.py. .env files will be accessed by docker-compose.yml to set environment variables inside respective containers
  • add ssl cert provision for gunicorn

…cker-compose.yml and commented school_metric_data.sql content as it was just for testing as confirmed by Partha (Pardhu448)
…s sliently (without user intervention/prompts). If arguments are missing it will prompt user for inputs
…py. .env files will be accessed by docker-compose.yml to set environment variables inside respective containers
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