Skip to content

DOC-2289: Emphasize required setup steps for remote read replicas (BYOVPC)#628

Open
micheleRP wants to merge 1 commit into
mainfrom
DOC-2289-rrr-byovpc-required-steps
Open

DOC-2289: Emphasize required setup steps for remote read replicas (BYOVPC)#628
micheleRP wants to merge 1 commit into
mainfrom
DOC-2289-rrr-byovpc-required-steps

Conversation

@micheleRP

@micheleRP micheleRP commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

What

Restructures the BYOC remote read replicas (RRR) page to make the required setup steps stand out, so users stop skipping the cluster-linking step.

Ref: DOC-2289 (Zendesk 7005, customer escalation)

Why

A customer creating an RRR topic in a BYOVPC deployment hit UNKNOWN_SERVER_ERROR: Unable to perform requested topic operation. Two required steps were documented but not prominent, so they were missed:

  1. Granting the reader cluster's service account read access to the source cluster's storage bucket.
  2. Linking the clusters via read_replica_cluster_ids. The customer created the topic but never linked, and it was silently missed (caught only by an engineer reviewing the thread, not by alerting).

Changes

All in modules/get-started/pages/cluster-types/byoc/remote-read-replicas.adoc:

  • Added an IMPORTANT callout before Prerequisites listing the three required steps up front, including "Creating the topic alone does not link the clusters."
  • Renamed "Configure remote read replica" to "Link the reader cluster to the source cluster" so the required action is clear in the heading and TOC.
  • Added the rationale for linking (safe upgrade ordering + source-deletion protection) and a WARNING that creating the topic does not link the clusters.
  • Added section anchors (byovpc-storage-permissions, link-clusters, create-remote-read-replica-topic) and updated the cross-region xref to use the new anchor.

Open question (asked in DOC-2289, pending reply)

Does source-cluster deletion protection trigger on the RRR topic existing or on the read_replica_cluster_ids link? The new WARNING attributes it to the link; the existing NOTE attributes it to the topic. These will be reconciled once Sarah/engineering confirms, in a follow-up commit on this PR.

Not included (deferred)

Discoverability / navigation changes for BYOVPC content (ask #2 in the ticket) are held pending scope confirmation from the reporter.

Preview pages

🤖 Generated with Claude Code

…OVPC)

Restructure the BYOC remote read replicas page so the required setup steps
stand out. Adds an up-front required-steps callout, renames the linking
section, explains why linking matters, and warns that creating the topic
alone does not link the clusters. Adds section anchors and updates the
cross-region xref to use one.

Ref: DOC-2289 (Zendesk 7005, customer escalation)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@micheleRP micheleRP requested a review from a team as a code owner July 1, 2026 21:39
@netlify

netlify Bot commented Jul 1, 2026

Copy link
Copy Markdown

Deploy Preview for rp-cloud ready!

Name Link
🔨 Latest commit d7f2f6a
🔍 Latest deploy log https://app.netlify.com/projects/rp-cloud/deploys/6a4588fdad06c300084d3b17
😎 Deploy Preview https://deploy-preview-628--rp-cloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Jul 1, 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: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c764649b-0897-4f9a-be3f-bf8cbd31a28d

📥 Commits

Reviewing files that changed from the base of the PR and between d007c7f and d7f2f6a.

📒 Files selected for processing (1)
  • modules/get-started/pages/cluster-types/byoc/remote-read-replicas.adoc

📝 Walkthrough

Walkthrough

This documentation-only change updates the remote read replicas BYOC page. It adds an IMPORTANT callout listing prerequisite steps (BYOVPC storage permissions, cluster linking via read_replica_cluster_ids, then topic creation), introduces a new "Link the reader cluster to the source cluster" section describing upgrade ordering and deletion protection behaviors, adds section anchors, and corrects a cross-region instruction's link reference to the topic creation command.

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

Changes

Area Description
Prerequisites callout New IMPORTANT block listing required setup steps for remote read replicas
New section "Link the reader cluster to the source cluster" explaining linking requirements and effects
Anchors Added byovpc-storage-permissions and create-remote-read-replica-topic anchors
Link fix Corrected cross-region reference link to local anchor

Related PRs: None identified.

Suggested labels: documentation

Suggested reviewers: None identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the change well but does not follow the required template sections or include the issue link, review deadline, and checks list. Rewrite the PR description to match the template: add the issue URL, review deadline, page preview section, and the checklist items.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main documentation change about emphasizing required BYOVPC remote read replica setup steps.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch DOC-2289-rrr-byovpc-required-steps

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

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.

1 participant