Skip to content

Update connection string to new cluster - #2881

Merged
dwwoelfel merged 1 commit into
mainfrom
update-conn-string
Aug 25, 2026
Merged

Update connection string to new cluster#2881
dwwoelfel merged 1 commit into
mainfrom
update-conn-string

Conversation

@dwwoelfel

Copy link
Copy Markdown
Contributor

Updates the prod connection url to use the new cluster.

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 662e51b1-e1a9-4905-b244-47120fc404f1

📥 Commits

Reviewing files that changed from the base of the PR and between 37d7a22 and afb9110.

📒 Files selected for processing (1)
  • server/scripts/prod_connection_string.sh

Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review.


📝 Walkthrough

Walkthrough

The production connection string script now defaults to the instant-8 RDS cluster. The --cluster-id argument remains available for explicit overrides.

Changes

Production connection string

Layer / File(s) Summary
Update default cluster identifier
server/scripts/prod_connection_string.sh
The default cluster identifier changes from aurora-instant-cluster to instant-8. Explicit --cluster-id input remains supported.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to afb91

This PR updates the production connection URL to the new cluster in one localized change, with no actionable merge-blocking risk remaining beyond normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: updating the production connection string to use the new cluster.
Description check ✅ Passed The description directly explains that the production connection URL now uses the new cluster.
Docstring Coverage ✅ Passed 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 1…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

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 1 files.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@dwwoelfel
dwwoelfel merged commit 4c3a12f into main Aug 25, 2026
34 checks passed
@dwwoelfel
dwwoelfel deleted the update-conn-string branch August 25, 2026 23:54
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.

2 participants