Skip to content

RDSC-6239: Correct database PrivateLink setup instructions - #4294

Merged
mich-elle-luna merged 4 commits into
mainfrom
RDSC-6239-aurora-networking
Oct 21, 2026
Merged

mich-elle-luna merged 4 commits into
mainfrom
RDSC-6239-aurora-networking

Conversation

@ZdravkoDonev-redis

@ZdravkoDonev-redis ZdravkoDonev-redis commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Correct the database PrivateLink setup instructions and show the recommended connection path.

Changes

  • Use TCP on the database port, with PostgreSQL port 5432 beside the settings.
  • Give the NLB its own security group. Split inbound and outbound instructions.
  • Allow database connections and TCP health checks from the NLB security group.
  • Replace "static IP" with the current writer address. Link to the existing failover Lambda.
  • State that the target Redis Cloud Pro database must already exist.
  • Move the RDS Proxy diagram to the RDS Proxy page. Keep its PostgreSQL limitation visible.
  • Add an editable SVG for the direct NLB connection and Lambda updates.

Read The Diagram

  • Solid arrows: RDI connects through PrivateLink and the NLB to the database writer.
  • Dashed arrows: RDS events reach SNS, which invokes Lambda.
  • Lambda resolves the writer endpoint and updates the target group.
  • Lambda does not carry database traffic.

The diagram follows the existing Redis Lambda module. The text uses short instructions and separates database connections from address updates.

Validation

  • git diff --check passed.
  • SVG XML and image references passed. Rendered SVG visually checked.
  • Focused Hugo rendering passed with the repository image hook, parser settings, and tabs (Hugo 0.154.2).
  • Verified the proxy image appears only on the proxy page and the Lambda anchor still works.
  • Full-site local build was stopped before completion. The PR site build remains a separate check.
  • AWS setup walkthrough and RDI connectivity qualification remain pending. Keep this PR in draft until those steps pass.

Ticket: RDSC-6239. SQL grants: #4293.


Note

Low Risk
Documentation and diagram assets only; no application or infrastructure code changes.

Overview
Corrects Redis Cloud RDI PrivateLink setup docs so they match the recommended path: TCP to the database writer through a dedicated NLB security group, with optional Lambda keeping the IP target group current after RDS failover.

In setup.md, the guide now tells readers to create the target Redis Cloud Pro database first, swaps the RDS/Aurora architecture image for a new SVG that separates database traffic (solid) from SNS → Lambda target-group updates (dashed), and rewrites EC2 and RDS/Aurora NLB steps—Custom TCP and port 5432 for PostgreSQL, NLB-specific target group docs, TCP health checks, writer DNS instead of a “static IP,” and inbound rules sourced from the NLB security group rather than open HTTP. The RDS/Aurora tab frames Lambda as address maintenance, not a proxy for query traffic, and tightens the RDS Proxy callout.

On rds-proxy.md, the former shared Aurora diagram moves here with an explicit non-PostgreSQL note for the proxy alternative.

Reviewed by Cursor Bugbot for commit f9d3c3e. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions

github-actions Bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

🧠 Redis Memory

Found 8 related items from repository history:

Memory updated at f9d3c3e

@github-actions

Copy link
Copy Markdown
Contributor

@ZdravkoDonev-redis
ZdravkoDonev-redis marked this pull request as ready for review October 19, 2026 18:24
@ZdravkoDonev-redis

Copy link
Copy Markdown
Collaborator Author

@andy-stark-redis Please review and if you're OK directly merge. Another PR which addresses feedback about our docs.

Also the CI stage that fails - I think it's not cause by my PR changes?

@mich-elle-luna mich-elle-luna left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thank you!

@mich-elle-luna
mich-elle-luna merged commit 07c8b52 into main Oct 21, 2026
100 of 102 checks passed
@mich-elle-luna
mich-elle-luna deleted the RDSC-6239-aurora-networking branch October 21, 2026 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants