search

DoSomething / infrastructure

🐄 DoSomething.org's infrastructure, managed by Terraform.
MIT License
3 stars 2 forks source link

Removing backup window to allow RDS to manage this #277

Closed blisteringherb closed 3 years ago

blisteringherb commented 3 years ago

What's this PR do?

This pull request removes the backup window variable to allow RDS to manage this. The apply errored out because the window overlaps the maintenance window. At the moment we don't need to have fine-grained control over that, and we can just let the existing window stand.

How should this be reviewed?

...

Any background context you want to provide?

https://app.terraform.io/app/dosomething/workspaces/quasar/runs/run-Cvt1FajKA1u7BvNn

Relevant tickets

References Pivotal #174829369.

Checklist

blisteringherb commented 3 years ago

@DFurnes it looks like the check for dosomething-prod is taking an exceptionally long time. Have you run into something like this before? The voting-app check failed the last time on this PR, and I tried to restart it but got an error.

DFurnes commented 3 years ago

I haven't run into that issue before. It looks like a request 503'd for that particular plan.

I tried pushing an empty commit to re-run these plans & see if that fixes things.