Skip to content

SIMSBIOHUB-850: Update backup schedules and retention#344

Merged
dylanrogowsky-oxd merged 1 commit intodevfrom
SIMSBIOHUB-850
Feb 27, 2026
Merged

SIMSBIOHUB-850: Update backup schedules and retention#344
dylanrogowsky-oxd merged 1 commit intodevfrom
SIMSBIOHUB-850

Conversation

@dylanrogowsky-oxd
Copy link
Collaborator

@dylanrogowsky-oxd dylanrogowsky-oxd commented Feb 27, 2026

Links to Jira Tickets

Description of Changes

  • Crunchy Postgres backup schedules and retention: Refactored the infrastructure/crunchy-db Helm chart to use four pgBackRest repos (hourly, daily, weekly, monthly) with distinct schedules and retention per environment (dev/test/prod).
  • Chart structure: Inlined the PostgresCluster and related templates (no longer using the crunchy-postgres subchart); updated Chart.yaml (removed subchart dependency, added appVersion); removed Chart.lock.
  • New/updated templates: Added PostgresCluster.yaml, _helpers.tpl, _s3.tpl, and s3Secret.yaml; backup config now drives schedules and retention from pgBackRest.repos in values.
  • Values: Restructured base values.yaml and environment overrides (values-dev.yaml, values-test.yaml, values-prod.yaml) so each environment defines four repos with path, full schedule, retention count, and retention type. Data source path updated to use the daily tier (e.g. /biohub/dev/daily) for restore/standby.

Testing Notes

  • Deploy the crunchy-db chart in a dev/test namespace and confirm the PostgresCluster and repo-host pods come up; verify pgBackRest cron jobs/schedules for the four repos.
  • Optionally run a manual full backup and confirm objects appear under the expected S3 paths (e.g. /biohub/<env>/hourly, /biohub/<env>/daily, etc.).
  • If testing restore/standby, set dataSource.enabled: true and ensure the daily repo path and stanza match the backup location.

@dylanrogowsky-oxd dylanrogowsky-oxd added the Do Not Merge PR should not be merged label Feb 27, 2026
@dylanrogowsky-oxd dylanrogowsky-oxd added the Ready For Review PR is ready for review label Feb 27, 2026
@dylanrogowsky-oxd dylanrogowsky-oxd marked this pull request as ready for review February 27, 2026 00:23
@dylanrogowsky-oxd dylanrogowsky-oxd force-pushed the SIMSBIOHUB-850 branch 2 times, most recently from f85c49f to 92eec3b Compare February 27, 2026 00:48
@dylanrogowsky-oxd dylanrogowsky-oxd removed the Do Not Merge PR should not be merged label Feb 27, 2026
@dylanrogowsky-oxd dylanrogowsky-oxd changed the title Update backup schedules and retention SIMSBIOHUB-850: Update backup schedules and retention Feb 27, 2026
Copy link
Collaborator

@mauberti-bc mauberti-bc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Update backup schedules and retention
@sonarqubecloud
Copy link

@dylanrogowsky-oxd dylanrogowsky-oxd merged commit f1c82ff into dev Feb 27, 2026
18 checks passed
@dylanrogowsky-oxd dylanrogowsky-oxd deleted the SIMSBIOHUB-850 branch February 27, 2026 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready For Review PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants