fix: switch backup CronJob from distopia-db-r to distopia-db-rw - #207
Conversation
distopia-db-r ("any" instance) consistently refused pg_dump's connection
in real production (Connection refused, reproduced across the scheduled
run and multiple manual retries), while distopia-db-rw kept working fine
for the app the entire time. Ruled out NetworkPolicy (same-namespace
traffic was always allowed, before and after the cnpg-system exception)
and resource pressure (kubectl top showed the instance nowhere near its
limits) as causes -- root cause on the -r Service specifically remains
unclear, but -rw is proven reliable, so use it instead. Revisit if
instances is ever raised above 1 for HA.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Preview deployed to: https://ThunLights.github.io/distopia/storybook/pr-preview-207 |
|
Warning Review limit reachedNext included review available in 23 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe PostgreSQL backup CronJob now connects to ChangesBackup endpoint
Estimated code review effort: 2 (Simple) | ~5 minutes Merge Risk: ⚪ Minimal · up to The backup job now targets the database endpoint that has been reliable in this environment. No actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Preview deployed to: https://ThunLights.github.io/distopia/storybook/pr-preview-207 |
Summary
distopia-db-r("any" instance Service) consistently refusedpg_dump's connection in real production (Connection refused, reproduced across the scheduled run and multiple manual retries), whiledistopia-db-rwkept working fine for the app the entire time.cnpg-systemexception) and resource pressure (kubectl topshowed the instance nowhere near its CPU/memory limits) as causes.-rService specifically remains unclear, but-rwis proven reliable in this environment, sok8s/db/backup-cronjob.yaml'sDB_HOSTnow points at it instead. Revisit-rvs-rwifinstancesis ever raised above 1 for HA.Test plan
kubectl create job --from=cronjob/distopia-db-backup distopia-db-backup-manual -n distopiacompletes successfully againstdistopia-db-rw0 18 * * *) also completes successfully🤖 Generated with Claude Code
Summary by CodeRabbit