[terraform] add backups to self-host aws rds
ClosedPublic
Actions

Authored by will on Mon, Sep 23, 6:31 AM.

Details

Reviewers

bartek
varun

Commits

rCOMMae1d18eb5546: [terraform] add backups to self-host aws rds

Summary

A previous commit for this had to be reverted. AWS randomly chooses maintainence window times and my selected maintainence window and backup window overlapping, causing the terraform apply to fail.

This diff ensures they are specified differently.

Test Plan

terraform apply. Ensured backup modifications were completed

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

will created this revision.Mon, Sep 23, 6:31 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMon, Sep 23, 6:32 AM

Harbormaster completed remote builds in B31776: Diff 44423.Mon, Sep 23, 6:49 AM

will requested review of this revision.Mon, Sep 23, 6:49 AM

bartek accepted this revision.Mon, Sep 23, 7:46 AM

bartek added inline comments.

services/terraform/self-host/aws_db.tf
49–50 ↗	(On Diff #44423)	These are UTC?

This revision is now accepted and ready to land.Mon, Sep 23, 7:46 AM

will added inline comments.Mon, Sep 23, 7:51 AM

services/terraform/self-host/aws_db.tf
49–50 ↗	(On Diff #44423)	Yep!

Closed by commit rCOMMae1d18eb5546: [terraform] add backups to self-host aws rds (authored by will). · Explain WhyMon, Sep 23, 10:04 PM

This revision was automatically updated to reflect the committed changes.

will added a commit: rCOMMae1d18eb5546: [terraform] add backups to self-host aws rds.

Revision Contents
Changeset List

Path

Size

services/

terraform/

self-host/

aws_db.tf

37 lines

Diff 44452

View Options

services/terraform/self-host/aws_db.tf

[terraform] add backups to self-host aws rdsClosedPublicActions