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

Orchestrate infra #69

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

Orchestrate infra #69

wants to merge 14 commits into from

Conversation

sonlinux
Copy link

In this PR I do the following

  • Add dynamic docker image
  • Add all services the project depends on as docker services
  • add base compose file to host the stack
  • add instructions to build the project locally and in production
  • Refine the layout of Django code and infrastructure
  • Update Sendgrid API code
  • Send most environment variables to docker-compose
  • Add Nginx with SSL support where necessary
  • Add Postgres DB from docker service
  • Manage CPU and RAM consumption within the stack definitions
  • Build with Debian based python=3.6.8
  • Django 2.2.10 conflicts as this module is deprecated,

👉 https://github.com/MicroPyramid/opensource-job-portal/blob/master/peeldb/models.py#L8

  • Hence resolving this means we can bump to use Django 2.X
  • with this in effect: This PR ships with Django==1.11.17

TODO { these will ship in separate PR after we merge this one }

Add .env template file to house all environment variables
Add makefile for more dynamic operations to container services
Add version bump support for quick changelog tagging
Build with Django==2.2.10

@sonlinux sonlinux requested review from ashwin31, manasatmicro and chaitu210 and removed request for ashwin31 and manasatmicro February 19, 2020 11:59
Copy link
Member

@ashwin31 ashwin31 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix conflicts

.gitignore Outdated Show resolved Hide resolved
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.

2 participants