Flip apps (including website/) to using managed postgres database#2631
Flip apps (including website/) to using managed postgres database#2631Will-Howard wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR updates nine Kubernetes service definitions to use a managed Postgres connection string for their database configuration. The affected services—bluedot-app-template, bluedot-frontend-example, bluedot-website, bluedot-website-production, bluedot-editor, bluedot-meet, bluedot-availability, bluedot-course-demos, and bluedot-speed-review—now reference Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
Greptile SummaryThis PR migrates 9 production services from the in-cluster CloudNativePG instance (
Confidence Score: 5/5Safe to merge — the change is a uniform config swap across all app services, the shadow sync service already keeps the target database populated, and no service logic is affected. All 9 affected services receive the same straightforward substitution. The managed database was already being kept in sync by the shadow pg-sync instance, so no data gap can occur. The primary pg-sync and Keycloak are correctly left untouched as part of the documented two-step migration. No logic errors or missing cases were found. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
AT[Airtable]
subgraph "Before PR"
PGS1[pg-sync-service]
OLD_DB[(airtableSyncPg\nin-cluster)]
APPS1[All app services\nwebsite, editor, meet, etc.]
AT --> PGS1 --> OLD_DB --> APPS1
end
subgraph "After PR"
PGS2[pg-sync-service\nprimary]
PGS_SHADOW[pg-sync-shadow]
OLD_DB2[(airtableSyncPg\nin-cluster)]
NEW_DB[(appPg\nmanaged Vultr)]
APPS2[All app services\nwebsite, editor, meet, etc.]
KC[Keycloak]
KC_DB[(keycloakPg)]
AT --> PGS2 --> OLD_DB2
AT --> PGS_SHADOW --> NEW_DB --> APPS2
KC --> KC_DB
end
Reviews (1): Last reviewed commit: "Swap apps to managed postgres" | Re-trigger Greptile |
Description
We have a new managed postgres database provisioned (#2572) and synced to Airtable (#2624). This PR flips all consumers to using the new database. I am doing all at once to reduce the risk of data inconsistencies due to different apps reading from different databases.
Because both postgres instances are synced via pg-sync-service, this should be a zero downtime change.
Issue
#2580
Developer checklist