Skip to content

Conversation

@cthumuluru-crdb
Copy link
Contributor

@cthumuluru-crdb cthumuluru-crdb commented Jan 5, 2026

Previously, loqrecovery tests would fail when DRPC was enabled because DRPC returns codes.Unknown for connection errors, while gRPC returns codes.Unavailable. DRPC version used in this patch has a fix to return codes.Unavailable for connection errors.

Fixes: #159372
Epic: CRDB-55382
Release note: None

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@cthumuluru-crdb cthumuluru-crdb added the do-not-merge bors won't merge a PR with this label. label Jan 5, 2026
@cthumuluru-crdb cthumuluru-crdb marked this pull request as ready for review January 5, 2026 08:22
@cthumuluru-crdb cthumuluru-crdb requested review from a team as code owners January 5, 2026 08:23
@Nukitt
Copy link
Contributor

Nukitt commented Jan 9, 2026

LGTM!

Previously, loqrecovery tests would fail when DRPC was enabled because
DRPC returns codes.Unknown for connection errors, while gRPC returns
codes.Unavailable. DRPC version used in this patch has a fix to return
codes.Unavailable for connection errors.

Fixes: cockroachdb#159372
Epic: CRDB-55382
Release note: None
@cthumuluru-crdb cthumuluru-crdb removed the do-not-merge bors won't merge a PR with this label. label Jan 12, 2026
@cthumuluru-crdb
Copy link
Contributor Author

bors r+

@craig craig bot merged commit de16d16 into cockroachdb:master Jan 12, 2026
26 checks passed
@craig
Copy link
Contributor

craig bot commented Jan 12, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

kvserver: Tests in loqrecovery fail with DRPC enabled

3 participants