Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

kvserver: never mark replicas as unreachable in TestFlowControl.* #142498

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kvoli
Copy link
Collaborator

@kvoli kvoli commented Mar 7, 2025

The TestFlowControl.* tests don't expect replicas to blip in and out
of StateProbe as a result of the testing environment or timing
conditions. Prohibit this via setting RaftReportUnreachableBypass.

Fixes: #142413
Fixes: #142442
Fixes: #142450
Fixes: #142465
Fixes: #142464
Fixes: #142463
Fixes: #142462
Fixes: #142451
Release note: None

@kvoli kvoli self-assigned this Mar 7, 2025
Copy link

blathers-crl bot commented Mar 7, 2025

Your pull request contains more than 1000 changes. It is strongly encouraged to split big PRs into smaller chunks.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@kvoli kvoli force-pushed the 250307.rac-mark-unreachable-test-flow-control branch from dffcf92 to e3c5710 Compare March 7, 2025 22:57
@kvoli kvoli requested review from pav-kv and stevendanna March 7, 2025 23:04
@kvoli kvoli marked this pull request as ready for review March 7, 2025 23:04
@kvoli kvoli requested a review from a team as a code owner March 7, 2025 23:04
The `TestFlowControl.*` tests don't expect replicas to blip in and out
of `StateProbe` as a result of the testing environment or timing
conditions. Prohibit this via setting `RaftReportUnreachableBypass`.

Fixes: cockroachdb#142413
Fixes: cockroachdb#142442
Fixes: cockroachdb#142450
Fixes: cockroachdb#142465
Fixes: cockroachdb#142464
Fixes: cockroachdb#142463
Fixes: cockroachdb#142462
Fixes: cockroachdb#142451
Release note: None
@kvoli kvoli force-pushed the 250307.rac-mark-unreachable-test-flow-control branch from e3c5710 to cf225dc Compare March 9, 2025 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment