NASA-IMPACT / veda-backend

Backend services for VEDA
Other
13 stars 5 forks source link

fix: update condition for setting storage_encrypted #326

Closed botanical closed 8 months ago

botanical commented 8 months ago

Issue

Link to relevant GitHub issue

What?

Why?

Testing?

anayeaye commented 8 months ago

RE Testing?

We will have some evidence that the change is safe for the existing dev and staging stacks in the pr deploy action cdk output which will alert if a change is caused that would require replacing the rds instance (which would appropriately fail if we tried to deploy the update because of our deletion protection setting on the instance). The pre deploy action cdk diff against the dev shows that the change doesn't impact that instance.

The staging stack uses the DatabaseInstanceFromSnapshot construct so we should check that diff on our next dev>main release.

anayeaye commented 8 months ago

@botanical you mentioned updating docs, I think, if anything we could add a line to the database snapshots of our wiki section with the links you gathered in the subject of this PR

botanical commented 8 months ago

@botanical you mentioned updating docs, I think, if anything we could add a line to the database snapshots of our wiki section with the links you gathered in the subject of this PR

Thanks, @anayeaye ! Wiki has been updated https://github.com/NASA-IMPACT/veda-architecture/wiki/DS:-Infrastructure-Maintenance-&-Upgrades#database-snapshots