Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DDLS-23 create weekly integration restore job #1588

Merged
merged 2 commits into from
Jun 3, 2024
Merged

DDLS-23 create weekly integration restore job #1588

merged 2 commits into from
Jun 3, 2024

Conversation

jamesrwarren
Copy link
Contributor

@jamesrwarren jamesrwarren commented May 14, 2024

Purpose

Run our restore on integration on a weekly basis to ensure it's fit for purpose in case of disaster.

Fixes DDLS-23

Approach

Checking for a point in time recovery and then a restore from a cross account snapshot.

The whole process is very very slow as that's how aws backups and restores work. The alternative is to pgdump and restore the data but we don't do that for real restores as this doesn't encrypt our data at rest in the backup account and doesn't allow for point in time recovery in the main account.

So anyway the approach here was to create some new roles in org infra and allow our preprod oidc role to assume them. We then run both restores back to back.

Learning

Everything about this takes ages to test so there's a lot of waiting for things to complete.. As such I have managed to test a full run of the cross acct backup to integration from the github pipeline and pit recovery from local but haven't run them both back to back in the pipeline. It ought to work though and we can fix forward if not.

Checklist

  • I have performed a self-review of my own code
  • I have updated documentation (Confluence/ADR/tech debt doc) where relevant
  • I have added tests to prove my work
  • The product team have approved these changes
  • I have checked my work for potential security issues and refered to the OWASP top 10

Frontend

  • I have run an in-browser accessibility test (e.g. WAVE, Lighthouse)
  • There are no deprecated CSS classes noted in the profiler
  • Translations are used and the profiler doesn't identify any missing
  • Any links or buttons added are screen reader friendly and contextually complete
  • If adding GA events, I have updated or checked the existing category or label values

@jamesrwarren jamesrwarren force-pushed the DDLS-23 branch 6 times, most recently from 52a02d8 to 3f4a16f Compare May 15, 2024 13:01
@jamesrwarren jamesrwarren changed the title DDLS-23 create monthly integration restore job DDLS-23 create weekly integration restore job May 15, 2024
@jamesrwarren jamesrwarren force-pushed the DDLS-23 branch 3 times, most recently from 677c08e to 5f4e3c1 Compare May 15, 2024 13:22
@jamesrwarren jamesrwarren marked this pull request as ready for review May 15, 2024 13:30
@jamesrwarren jamesrwarren requested a review from a team as a code owner May 15, 2024 13:30
@jamesrwarren jamesrwarren merged commit 70b7be7 into main Jun 3, 2024
37 checks passed
@jamesrwarren jamesrwarren deleted the DDLS-23 branch June 3, 2024 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants