search

SolutionGuidance / psm

Welcome to the Medicare/Medicaid Provider Enrollment Screening Portal
http://projectpsm.org/
Other
26 stars 20 forks source link

Move to ansible for demo setup #1024

Closed jvasile closed 5 years ago

jvasile commented 6 years ago

This patch takes all the steps in the install.sh and recasts them as an ansible role. It replaces the original install.sh script with a couple lines that invoke ansible.

jvasile commented 6 years ago

Hmm... this is failing some manual testing. I'll fix and update it.

PaulMorris commented 6 years ago

@jvasile The install.sh doesn't have the latest steps to set up and seed the database (the Liquibase migrations). These are in INSTALL.md > Build and deploy the application > 4. Create database schema and initial data. So we'll need to add those to the ansible role.

PaulMorris commented 5 years ago

I added a commit that adds the liquibase db setup step. I haven't had a chance to test this yet.

frankduncan commented 5 years ago

I noticed that the "Seed Database" still cats seed.sql, so that can be removed.

Otherwise, this all looks straightforward and correct to me. I don't have an ubuntu or fedora machine handy, but we may want to create an issue to track the idea of making ansible the default way to install the app on jenkins in the future, otherwise this will only ever get tested when someone new onboards, and the likelihood of it going stale is probably pretty high.

PaulMorris commented 5 years ago

I've removed the "seed.sql" file from the "Seed Database" step. Will go ahead and merge since this is more up-to-date than the script it is replacing.

PaulMorris commented 5 years ago

I've opened Issue #1099 "Use ansible script for installing on Jenkins" to track that idea, as @frankduncan suggested.