Skip to content

tests: assert the multi-region precondition instead of assuming it #2

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.

Sign up for GitHub

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

Merged
eduralph merged 6 commits into from
Jul 12, 2026
Merged

tests: assert the multi-region precondition instead of assuming it #2

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
b6e1f39
tests: assert the multi-region precondition instead of assuming it
eduralph Jul 12, 2026
d3957c2
tests: locate both keys in one PD snapshot, and honour every PD endpoint
eduralph Jul 12, 2026
df2b30e
tests: hold the cross-region precondition per attempt, and bound ever…
eduralph Jul 12, 2026
9a34b0b
tests: fall through to the next PD endpoint on an unhealthy response,…
eduralph Jul 12, 2026
27f2651
tests: split the region at the key we need, instead of racing the spl…
eduralph Jul 12, 2026
37d6c8c
tests: give p0 a per-run key range so a stale boundary cannot satisfy it
eduralph Jul 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
192 changes: 189 additions & 3 deletions Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,10 @@ log = "0.4"
env_logger = "0.10"
serde_json = "1"
tokio = { version = "1", features = ["macros", "rt-multi-thread", "sync", "time"] }
# PD's HTTP API is the ground truth for region layout (tests/common/cluster.rs).
# rustls, not native-tls: PD is plain HTTP here, and this avoids dragging in an
# OpenSSL build just to read /pd/api/v1/regions.
reqwest = { version = "0.12", default-features = false, features = ["json", "rustls-tls"] }
# For the failpoint proof (tests/failpoint_gate.rs). Enabling `fail/failpoints`
# here activates the `after-prewrite` failpoint compiled into tikv-client (via
# Cargo feature unification), letting the test make a commit fail *after*
Expand Down
Loading
Loading