search

scientist-softserv / louisville-hyku

Other
0 stars 0 forks source link

add ENV variables to helm for postgres-backed fedora #196

Closed bkiahstroud closed 1 year ago

bkiahstroud commented 1 year ago

Story

Refs:

Follow up to https://github.com/scientist-softserv/louisville-hyku/pull/192

Expected Behavior Before Changes

Staging deploy does not have ENV variables for a postgres-backed Fedora database

Expected Behavior After Changes

Staging deploy has ENV variables for a postgres-backed Fedora database

bkiahstroud commented 1 year ago

Summary from discussion with Rob in standup:

These changes should NOT be merged/deployed. The shared staging Fedora instance should already be backed by postgres, so testing #192 there is impossible anyway. Also, the live site is not deployed via helm charts, so these changes are unnecessary. If needed, we can test #192 by requesting a copy of the production data and running the rake task locally.