hackoregon / transportation-system-backend

2018 repo for the transportation api backend
MIT License
8 stars 2 forks source link

Create a Deploy Ready version of Crash API #24

Closed bhgrant8 closed 6 years ago

bhgrant8 commented 6 years ago

Certain config changes will need to be made in order to enable our API to be deployed to the AWS cluster. As our Crash API is in a functional form exposing the basic tables, it would be beneficial for us to align with DevOps needs. I would prefer to deploy early and then continue to build from the base point.

Our work here should closely align with this ticket from ops: https://github.com/hackoregon/civic-devops/issues/4

Issue here is to track what we need to track our deliverable from a ops/platform standpoint to get something live that follows our CI/CD deploy strategy.

Base Requirements:

Breaking down the last requirement into a bit more detail:

znmeb commented 6 years ago

I'm going to propose we use https://github.com/hackoregon/transportation-system-backend/tree/znmeb-merge-new-database as the test case for this, now that it's working locally on my workstation.

bhgrant8 commented 6 years ago

Hey @znmeb, I have done most of the work for this in a branch. I still need to review your branch and see how they work together, which I have not had time to do as of yet.

Ultimately we are not looking to move far away from our development/deploy strategy of last year, or to a system which is overly complex, so this will be a consideration and has been in the initial development of our container and repo.

znmeb commented 6 years ago

which branch are you in? I can just fix all the bugs in that the same way I did in mine. ;-)

bhgrant8 commented 6 years ago

@znmeb reviewed your branch, only one real issue I saw, maybe a reason for it? Let's focus on that for the moment. I have a game plan here.

znmeb commented 6 years ago

What was the issue? Is it flagged in the review comments?

bhgrant8 commented 6 years ago

closing as stale