cockroachdb / cockroachdb/cockroach

crosscluster/physical: run maybeSetupPCRStandbyReader upgrade first

Open
#143,917 1 comment 0 reactions 1 assignee Claimed by @msbutler View on GitHub
A-disaster-recovery branch-release-25.2 C-bug P-1 T-disaster-recovery
Dominant language
Go
Stars
32.5k
Forks
4.1k
PR merge metrics
PR metrics pending

Description

Currently, there are several migration that would fail if the pcr reader migration were ran first, which means `bootstrapCluster` is not idempotent. For example, `addRootUser` would fail as it writes to the external user and role tables. Instead, we ought to run `maybeSetupPCRStandbyReader` first and manually skip migrations that are not appropriate to run on the reader tenant.

Jira issue: CRDB-49159

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.