-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
I modified x.damsolrizier.sh in all environments (qa, staging, production) to use environment variables. |
@rstanonik - Oh, sorry I missed this comment before making my last commit. So, should I vendor the main damsolrizer script, but leave |
@rstanonik - one thing we could probably do is add a new environment variable for damspas in our environment variables repo that sets the proper |
d49809b
to
6a11b01
Compare
After speaking with @rstanonik and looking at the env vars repo, it seems @jhriv already solved this problem for us :) updating the PR accordingly :) |
6a11b01
to
4268e93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Thanks. 👍
👍 |
1 similar comment
👍 |
Fixes issues we're seeing with damspas deploys and NOT restarting damsolrizer.
Example output on a dry run of capistrano w/ this new task added:
Deployment test to QA
Damspas deployed with a running version of
damsolrizer
:Damspas re-deployed w/ this branch:
Note, the PID's change, and damsolrizer is still up and running
Local Checklist
master
branch?What does this PR do?
Adds a capistrano task to restart damsolrizer on deployment.
@ucsdlib/developers - please review