Files
Anton Antonov 4c4c3d967e test: improve consensus test suite (#10671)
* test: add a gated proxy for peer RPCs

Pause one selected internal request while other peer traffic continues.
Preserve payloads, metadata, deadlines, and cancellation so consensus
tests can control transfer timing without blocking unrelated requests.

Cover forwarding, independent gates, and cleanup with socket tests.

* test: connect peer proxies to consensus clusters

Let consensus tests route internal RPCs through request gates. Keep each
proxy alive across peer restarts so advertised addresses remain stable,
and close all proxies during test cleanup.

Wait for the upstream gRPC connection before returning from proxied
startup. Verify consensus progress during a held WAL-delta request,
recovery data, and restart behavior with both URI configuration modes.

* test: fix potentially misleading peer proxy method names

Explicitly state the guarantees, or lack of.

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>

* test: add support for hold_snapshot_download

Removes flakiness from snapshot-related consensus tests too

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>

* test: improve asserts when force deleting peer

Actually verify survivors recover and retain the expected data.
Making sure no data loss happens.

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>

* test: add OsError socket handling + explicit wal_delta tests

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>

* test: reject zero as a defined consensus leader

* test: recheck leader agreement on each poll

After a restart, the leader can change during election. Let the cluster
wait resample the leader on each poll and require agreement on a nonzero
leader before the snapshot test starts its transfer.

Keep explicit leader checks for existing callers, membership-size checks,
and the existing timeout. Cover election changes and offline peers.

* test: verify independent snapshot download gates

* test: use a positive peer connection deadline

* test: cover recovery after the removed source exits

* chore: add clarifying comment on timeout=0 usage

It's not obvious at first why it's like so.

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>

* test: share consensus response gates

Move response gates, their tests, and Raft decoding from the leader
removal proof into the base test infrastructure. Both removal scenarios
can then use the same successful-response check.

* test: support selective RPC blocking

Keep a removed source unaware of membership changes while its transfer
continues. Block its Raft traffic in both directions so election
attempts cannot disrupt survivor recovery.

* test: make source removal scenarios deterministic

Separate recovery after source exit from late data sent by a removed
source. Require a successful receiver response in the late scenario,
and retain complete data and replica-state checks in both cases.

* test: refactor timeouts and deadlines

* Cancellation happens after observing the intended phase, without an RPC deadline.
* Separate deadline tests cover held requests, upstream work, and held responses.

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>

* test: bound peer probes and removal requests

Give cluster probes and peer removal finite client timeouts so a stalled
HTTP request cannot leave the test waiting indefinitely.

* test: separate RPC release from termination

Keep the upstream handler blocked until the test releases it or the RPC
terminates. Use a separate termination event for cancellation assertions,
and release the handler during teardown instead of racing a fixture timer.

* test: use monotonic polling deadlines

Measure elapsed polling time with a monotonic clock so system clock
adjustments cannot shorten or extend the wait.

* test: allow more time to observe proxy events

Allow ten seconds for proxy observations and ordinary test requests.
Event and future waits still return as soon as they complete. Keep the
one-second expiry tests and document the HTTP deadline setup race.

* test: bound leader and replication requests

Limit how long leader lookup and transfer submission wait for an HTTP
response. A stalled submission must fail so the test can release its
transfer gates and clean up the peers.

* test: preserve readiness failures in diagnostics

Catch request failures while collecting cluster diagnostics, including
read timeouts. Report the original readiness failure instead of replacing
it with a diagnostic error.

* test: assert points calls for the correct collection

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>

* test: make sure check_cluster_size and check_leader cannot stall

Have an explicit timeout.

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>

* test: retry timeouts during initial leader lookup

Treat request timeouts as retryable while discovering the expected leader,
matching the subsequent leader and membership checks. Keep polling after
a transient timeout instead of aborting the cluster-status wait.

* test: ensure batch data is different

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>

---------

Signed-off-by: Anton Antonov <anton.synd.antonov@gmail.com>
2026-09-21 12:04:51 +03:00

1095 lines
42 KiB
Python

import json
import os
import re
import shutil
import signal
from subprocess import Popen
import time
from typing import Tuple, Callable, Dict, List, Optional
import requests
import socket
from contextlib import ExitStack, closing
from pathlib import Path
import pytest
from .assertions import assert_http_ok
from .peer_proxy import PeerProxy
WAIT_TIME_SEC = 30
RETRY_INTERVAL_SEC = 0.2
PROJECT_ROOT = Path(__file__).parent.parent.parent
# Tracks processes that need to be killed at the end of the test
processes: List['PeerProcess'] = []
busy_ports = {}
peer_proxies: Dict[int, PeerProxy] = {}
class PeerProcess:
def __init__(self, proc: Popen, http_port, grpc_port, p2p_port):
self.proc = proc
self.http_port = http_port
self.grpc_port = grpc_port
self.p2p_port = p2p_port
self.pid = proc.pid
self.proxy = peer_proxies.get(p2p_port)
def kill(self):
self.proc.kill()
self.proc.wait()
busy_ports.pop(self.http_port, None)
busy_ports.pop(self.grpc_port, None)
busy_ports.pop(self.p2p_port, None)
def interrupt(self):
self.proc.send_signal(signal.SIGINT)
self.proc.wait()
busy_ports.pop(self.http_port, None)
busy_ports.pop(self.grpc_port, None)
busy_ports.pop(self.p2p_port, None)
def _occupy_port(port):
if port in busy_ports:
raise Exception(f'Port "{port}" was already allocated!')
busy_ports[port] = True
return port
def kill_all_processes():
print()
while len(processes) > 0:
p = processes.pop(0)
try:
if is_coverage_mode():
print(f"Interrupting {p.pid}")
p.interrupt()
else:
print(f"Killing {p.pid}")
p.kill()
except Exception as e:
print(f"Cleanup error for {p.pid}: {e}")
# Keep advertised addresses alive through peer restarts within the test.
with ExitStack() as cleanup:
while peer_proxies:
_, proxy = peer_proxies.popitem()
cleanup.callback(busy_ports.pop, proxy.port, None)
cleanup.callback(busy_ports.pop, proxy.http_port, None)
cleanup.callback(proxy.close)
# Each pytest-xdist worker owns a disjoint slice of the port space, so concurrent
# workers never compete for the same port range. Ports stay below the Linux
# default ephemeral range (32768) so OS-assigned random sockets don't collide
# either. Slice size of 300 = 100 peer triples, which comfortably covers any
# single test even with dynamically added peers.
_PORT_SLICE_BASE = 20000
_PORT_SLICE_SIZE = 300
def _xdist_worker_index() -> int:
name = os.environ.get("PYTEST_XDIST_WORKER", "gw0")
if name.startswith("gw") and name[2:].isdigit():
return int(name[2:])
return 0
_WORKER_SLICE_START = _PORT_SLICE_BASE + _xdist_worker_index() * _PORT_SLICE_SIZE
_WORKER_SLICE_END = _WORKER_SLICE_START + _PORT_SLICE_SIZE
_next_port_in_slice = _WORKER_SLICE_START
def _reset_port_slice():
global _next_port_in_slice
_next_port_in_slice = _WORKER_SLICE_START
@pytest.fixture(autouse=True)
def every_test():
if processes or peer_proxies:
print(f"WARN: {len(processes)} leaked peer processes from previous test, cleaning")
kill_all_processes()
_reset_port_slice()
yield
kill_all_processes()
def get_port() -> int:
while True:
with closing(socket.socket(socket.AF_INET, socket.SOCK_STREAM)) as s:
# get random port assigned by the OS
s.bind(('', 0))
s.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
allocated_port = s.getsockname()[1]
# Reserve a ±2 buffer so a restart using `port=p.p2p_port` (which
# also occupies port+1, port+2) cannot collide with another live
# peer's randomly-allocated port.
if any((allocated_port + d) in busy_ports for d in range(-2, 3)):
continue
return allocated_port
def _try_bind_triple(base: int) -> bool:
sockets = []
try:
for offset in range(3):
s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
try:
s.bind(('', base + offset))
sockets.append(s)
except OSError:
return False
return True
finally:
for s in sockets:
s.close()
def get_port_triple() -> int:
# Allocate a contiguous triple (p2p, grpc, http) for a peer. Each xdist
# worker draws from its own slice, so the original cross-worker collision
# on `port+1` / `port+2` (which restart paths derive from p2p_port) cannot
# happen. Within the slice we still probe-bind() each candidate so
# unrelated processes occupying a slot are skipped, not deterministically
# crashed-into. Falls back to OS-assigned ports if the slice is exhausted.
global _next_port_in_slice
while _next_port_in_slice + 3 <= _WORKER_SLICE_END:
base = _next_port_in_slice
_next_port_in_slice += 3
if _try_bind_triple(base):
return base
return _get_port_triple_from_os()
def _get_port_triple_from_os() -> int:
while True:
s0 = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
try:
s0.bind(('', 0))
base = s0.getsockname()[1]
s1 = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
s2 = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
try:
s1.bind(('', base + 1))
s2.bind(('', base + 2))
return base
except OSError:
continue
finally:
s1.close()
s2.close()
except OSError:
continue
finally:
s0.close()
def is_coverage_mode() -> bool:
return os.getenv("COVERAGE") == "1"
def get_env(p2p_port: int, grpc_port: int, http_port: int) -> Dict[str, str]:
env = os.environ.copy()
env["QDRANT__CLUSTER__ENABLED"] = "true"
env["QDRANT__CLUSTER__P2P__PORT"] = str(p2p_port)
env["QDRANT__SERVICE__HTTP_PORT"] = str(http_port)
env["QDRANT__SERVICE__GRPC_PORT"] = str(grpc_port)
env["QDRANT__LOG_LEVEL"] = "TRACE,raft::raft=info,actix_http=info,tonic=info,want=info,mio=info"
env["QDRANT__SERVICE__HARDWARE_REPORTING"] = "true"
# Fail peer when consensus state machine diverges from operation handlers
env["QDRANT__CLUSTER__CONSENSUS__SHADOW_STATE_MACHINE"] = "panic"
if is_coverage_mode():
env["LLVM_PROFILE_FILE"] = get_llvm_profile_file()
return env
def get_uri(port: int) -> str:
return f"http://127.0.0.1:{port}"
def get_peer_consensus_uri(p2p_port: int, use_peer_proxy: bool = False) -> str:
proxy = peer_proxies.get(p2p_port)
if proxy is None and use_peer_proxy:
while True:
proxy = PeerProxy(f"127.0.0.1:{p2p_port}")
# The peer's port triple is reserved but may not be listening yet.
if proxy.port not in busy_ports and proxy.http_port not in busy_ports:
break
proxy.close()
_occupy_port(proxy.port)
_occupy_port(proxy.http_port)
peer_proxies[p2p_port] = proxy
return proxy.uri if proxy is not None else get_uri(p2p_port)
def assert_project_root():
"""Deprecated: No longer needed as paths are resolved relative to __file__."""
pass
def get_qdrant_exec() -> str:
if is_coverage_mode():
qdrant_exec = PROJECT_ROOT / "target" / "llvm-cov-target" / "debug" / "qdrant"
else:
qdrant_exec = PROJECT_ROOT / "target" / "debug" / "qdrant"
return str(qdrant_exec)
def get_llvm_profile_file() -> str:
# %p (per-PID) avoids %m's partial-merge corruption when peers are SIGKILLed under -n auto.
llvm_profile_file = PROJECT_ROOT / "target" / "llvm-cov-target" / "qdrant-consensus-tests-%p.profraw"
return str(llvm_profile_file)
def get_pytest_current_test_name() -> str:
# https://docs.pytest.org/en/latest/example/simple.html#pytest-current-test-environment-variable
return os.environ.get('PYTEST_CURRENT_TEST').split(':')[-1].split(' ')[0]
def init_pytest_log_folder() -> str:
test_name = get_pytest_current_test_name()
log_folder = f"consensus_test_logs/{test_name}"
if not os.path.exists(log_folder):
os.makedirs(log_folder)
return log_folder
# Starts a peer and returns its api_uri
def start_peer(peer_dir: Path, log_file: str, bootstrap_uri: str, port=None, extra_env=None, reinit=False, uris_in_env=False, use_peer_proxy=False) -> str:
if extra_env is None:
extra_env = {}
base_port = get_port_triple() if port is None else port
p2p_port = base_port + 0
_occupy_port(p2p_port)
grpc_port = base_port + 1
_occupy_port(grpc_port)
http_port = base_port + 2
_occupy_port(http_port)
test_log_folder = init_pytest_log_folder()
log_file = open(f"{test_log_folder}/{log_file}", "w")
this_peer_consensus_uri = get_peer_consensus_uri(p2p_port, use_peer_proxy)
print(f"Starting follower peer with bootstrap uri {bootstrap_uri},"
f" http: http://localhost:{http_port}/cluster, p2p: {p2p_port}")
args = [get_qdrant_exec()]
env = {
**get_env(p2p_port, grpc_port, http_port),
**extra_env
}
if p2p_port in peer_proxies:
env.update(peer_proxies[p2p_port].env)
if uris_in_env:
env["QDRANT_BOOTSTRAP"] = bootstrap_uri
env["QDRANT_URI"] = this_peer_consensus_uri
else:
args.extend(["--bootstrap", bootstrap_uri, "--uri", this_peer_consensus_uri])
if reinit:
args.append("--reinit")
# Wrap with systemd-run to throttle CPU to investigate issues
# wrapped_cmd = ["systemd-run", "--user", "--scope", "-p", "CPUQuota=20%", "--"] + args
# proc = Popen(wrapped_cmd, env=env, cwd=peer_dir, stdout=log_file)
proc = Popen(args, env=env, cwd=peer_dir, stdout=log_file)
processes.append(PeerProcess(proc, http_port, grpc_port, p2p_port))
if processes[-1].proxy is not None:
processes[-1].proxy.wait_for_peer_connection()
return get_uri(http_port)
# Starts a peer and returns its api_uri and p2p_uri
def start_first_peer(peer_dir: Path, log_file: str, port=None, extra_env=None, reinit=False, uris_in_env=False, use_peer_proxy=False) -> Tuple[str, str]:
if extra_env is None:
extra_env = {}
base_port = get_port_triple() if port is None else port
p2p_port = base_port + 0
_occupy_port(p2p_port)
grpc_port = base_port + 1
_occupy_port(grpc_port)
http_port = base_port + 2
_occupy_port(http_port)
test_log_folder = init_pytest_log_folder()
log_file = open(f"{test_log_folder}/{log_file}", "w")
bootstrap_uri = get_peer_consensus_uri(p2p_port, use_peer_proxy)
print(f"\nStarting first peer with uri {bootstrap_uri},"
f" http: http://localhost:{http_port}/cluster, p2p: {p2p_port}")
args = [get_qdrant_exec()]
env = {
**get_env(p2p_port, grpc_port, http_port),
**extra_env
}
if p2p_port in peer_proxies:
env.update(peer_proxies[p2p_port].env)
if uris_in_env:
env["QDRANT_URI"] = bootstrap_uri
else:
args.extend(["--uri", bootstrap_uri])
if reinit:
args.append("--reinit")
# Wrap with systemd-run to throttle CPU to investigate issues
# wrapped_cmd = ["systemd-run", "--user", "--scope", "-p", "CPUQuota=20%", "--"] + args
# proc = Popen(wrapped_cmd, env=env, cwd=peer_dir, stdout=log_file)
proc = Popen(args, env=env, cwd=peer_dir, stdout=log_file)
processes.append(PeerProcess(proc, http_port, grpc_port, p2p_port))
if processes[-1].proxy is not None:
processes[-1].proxy.wait_for_peer_connection()
return get_uri(http_port), bootstrap_uri
def start_cluster(tmp_path, num_peers, port_seed=None, extra_env=None, headers={}, uris_in_env=False, log_file_prefix="", use_peer_proxy=False):
"""Optionally route internal RPCs and snapshot downloads through each peer's proxy.
Proxies survive restarts on the same P2P port and close during test cleanup.
Client-facing REST and public gRPC keep their direct addresses.
"""
assert_project_root()
peer_dirs = make_peer_folders(tmp_path, num_peers)
# Gathers REST API uris
peer_api_uris = []
# Start bootstrap
(bootstrap_api_uri, bootstrap_uri) = start_first_peer(peer_dirs[0], f"{log_file_prefix}peer_0_0.log", port=port_seed,
extra_env=extra_env, uris_in_env=uris_in_env, use_peer_proxy=use_peer_proxy)
peer_api_uris.append(bootstrap_api_uri)
# Wait for leader
leader = wait_peer_added(bootstrap_api_uri, headers=headers)
port = None
# Start other peers
for i in range(1, len(peer_dirs)):
if port_seed is not None:
port = port_seed + i * 100
peer_api_uris.append(start_peer(peer_dirs[i], f"{log_file_prefix}peer_0_{i}.log", bootstrap_uri, port=port, extra_env=extra_env, uris_in_env=uris_in_env, use_peer_proxy=use_peer_proxy))
# Wait for cluster
wait_for_uniform_cluster_status(peer_api_uris, leader, headers=headers)
return peer_api_uris, peer_dirs, bootstrap_uri
def make_peer_folder(base_path: Path, peer_number: int) -> Path:
peer_dir = base_path / f"peer{peer_number}"
peer_dir.mkdir()
shutil.copytree(PROJECT_ROOT / "config", peer_dir / "config")
return peer_dir
def make_peer_folders(base_path: Path, n_peers: int) -> List[Path]:
peer_dirs = []
for i in range(n_peers):
peer_dir = make_peer_folder(base_path, i)
peer_dirs.append(peer_dir)
return peer_dirs
def get_cluster_info(peer_api_uri: str, headers={}) -> dict:
r = requests.get(f"{peer_api_uri}/cluster", headers=headers, timeout=10)
assert_http_ok(r)
res = r.json()["result"]
return res
def print_clusters_info(peer_api_uris: [str], headers={}):
for uri in peer_api_uris:
try:
# do not crash if the peer is not online
print(json.dumps(get_cluster_info(uri, headers=headers), indent=4))
except requests.exceptions.RequestException as error:
print(f"Can't retrieve cluster info for peer {uri}: {error}")
def fetch_highest_peer_id(peer_api_uris: [str]) -> str:
max_peer_id = 0
max_peer_url = None
for uri in peer_api_uris:
try:
# do not crash if the peer is not online
peer_id = get_cluster_info(uri)['peer_id']
if peer_id > max_peer_id:
max_peer_id = peer_id
max_peer_url = uri
except requests.exceptions.ConnectionError:
print(f"Can't retrieve cluster info for offline peer {uri}")
return max_peer_url
def get_collection_cluster_info(peer_api_uri: str, collection_name: str, headers={}) -> dict:
r = requests.get(f"{peer_api_uri}/collections/{collection_name}/cluster", headers=headers, timeout=10)
assert_http_ok(r)
res = r.json()["result"]
return res
def get_shard_transfer_count(peer_api_uri: str, collection_name: str) -> int:
info = get_collection_cluster_info(peer_api_uri, collection_name)
return len(info["shard_transfers"])
def get_collection_info(peer_api_uri: str, collection_name: str) -> dict:
r = requests.get(f"{peer_api_uri}/collections/{collection_name}")
assert_http_ok(r)
res = r.json()["result"]
return res
def get_collection_point_count(peer_api_uri: str, collection_name: str, exact: bool = False,
shard_key: Optional[str] = None, filter: Optional[dict] = None) -> int:
r = requests.post(f"{peer_api_uri}/collections/{collection_name}/points/count", json={"exact": exact, "shard_key": shard_key, "filter": filter})
assert_http_ok(r)
res = r.json()["result"]["count"]
return res
def print_collection_cluster_info(peer_api_uri: str, collection_name: str, headers={}):
print(json.dumps(get_collection_cluster_info(peer_api_uri, collection_name, headers=headers), indent=4))
def get_leader(peer_api_uri: str, headers={}) -> str:
r = requests.get(f"{peer_api_uri}/cluster", headers=headers, timeout=10)
assert_http_ok(r)
return r.json()["result"]["raft_info"]["leader"]
def check_leader(peer_api_uri: str, expected_leader: str, headers={}) -> bool:
try:
r = requests.get(f"{peer_api_uri}/cluster", headers=headers, timeout=10)
assert_http_ok(r)
leader = r.json()["result"]["raft_info"]["leader"]
correct_leader = leader == expected_leader
if not correct_leader:
print(f"Cluster leader invalid for peer {peer_api_uri} {leader}/{expected_leader}")
return correct_leader
except (requests.exceptions.ConnectionError, requests.exceptions.Timeout):
# the api is not yet available - caller needs to retry
print(f"Could not contact peer {peer_api_uri} to fetch cluster leader")
return False
def leader_is_defined(peer_api_uri: str, headers={}) -> bool:
try:
r = requests.get(f"{peer_api_uri}/cluster", headers=headers)
assert_http_ok(r)
leader = r.json()["result"]["raft_info"]["leader"]
return leader not in (None, 0)
except requests.exceptions.ConnectionError:
# the api is not yet available - caller needs to retry
print(f"Could not contact peer {peer_api_uri} to fetch leader info")
return False
def check_cluster_size(peer_api_uri: str, expected_size: int, headers={}) -> bool:
try:
r = requests.get(f"{peer_api_uri}/cluster", headers=headers, timeout=10)
assert_http_ok(r)
peers = r.json()["result"]["peers"]
correct_size = len(peers) == expected_size
if not correct_size:
print(f"Cluster size invalid for peer {peer_api_uri} {len(peers)}/{expected_size}")
return correct_size
except (requests.exceptions.ConnectionError, requests.exceptions.Timeout):
# the api is not yet available - caller needs to retry
print(f"Could not contact peer {peer_api_uri} to fetch cluster size")
return False
def all_nodes_cluster_info_consistent(peer_api_uris: [str], expected_leader: Optional[str] = None, headers={}) -> bool:
if expected_leader is None:
# Elections can change the leader between polls, especially after a restart.
if not peer_api_uris:
return False
try:
expected_leader = get_leader(peer_api_uris[0], headers=headers)
except (requests.exceptions.ConnectionError, requests.exceptions.Timeout):
print(f"Could not contact peer {peer_api_uris[0]} to fetch cluster leader")
return False
if expected_leader in (None, 0):
return False
expected_size = len(peer_api_uris)
for uri in peer_api_uris:
if check_leader(uri, expected_leader, headers=headers) and check_cluster_size(uri, expected_size, headers=headers):
continue
else:
return False
return True
def peers_have_version(peer_api_uris: [str]) -> bool:
for peer_api_uri in peer_api_uris:
try:
# Check versions in local telemetry of each peer
# Not using cluster level telemetry because it shows versions before
# they are fully propagated through consensus
r = requests.get(f"{peer_api_uri}/telemetry?details_level=3")
assert_http_ok(r)
cluster = r.json()["result"]["cluster"]
peers = cluster["peers"]
peer_metadata = cluster["peer_metadata"]
for peer_id in peers.keys():
# Peers without metadata are not listed
if peer_id not in peer_metadata:
return False
if peer_metadata[peer_id].get("version") is None:
return False
except requests.exceptions.ConnectionError:
print(f"Could not contact peer {peer_api_uri} to fetch versions")
return False
return True
def all_nodes_have_same_commit(peer_api_uris: [str]) -> bool:
commits = []
for uri in peer_api_uris:
try:
r = requests.get(f"{uri}/cluster")
assert_http_ok(r)
commits.append(r.json()["result"]["raft_info"]["commit"])
except requests.exceptions.ConnectionError:
print(f"Could not contact peer {uri} to fetch commit")
return False
return len(set(commits)) == 1
def all_nodes_respond(peer_api_uris: [str]) -> bool:
for uri in peer_api_uris:
try:
r = requests.get(f"{uri}/collections")
assert_http_ok(r)
except requests.exceptions.ConnectionError:
print(f"Could not contact peer {uri} to fetch collections")
return False
return True
def all_nodes_have_applied_same_commit(peer_api_uris: [str]) -> bool:
"""
Like `all_nodes_have_same_commit`, but also requires every peer to have
applied all of its committed entries. A peer that has just joined can share
the leader's commit index while its local state is still being replayed.
"""
commits = []
for uri in peer_api_uris:
try:
r = requests.get(f"{uri}/cluster")
assert_http_ok(r)
raft_info = r.json()["result"]["raft_info"]
except requests.exceptions.ConnectionError:
print(f"Could not contact peer {uri} to fetch commit")
return False
if raft_info["pending_operations"] != 0:
return False
commits.append(raft_info["commit"])
return len(set(commits)) == 1
def all_peers_are_voters(peer_api_uris: [str]) -> bool:
try:
for uri in peer_api_uris:
if not get_cluster_info(uri)["raft_info"]["is_voter"]:
return False
return True
except requests.exceptions.ConnectionError:
return False
def collection_exists_on_all_peers(collection_name: str, peer_api_uris: [str]) -> bool:
for uri in peer_api_uris:
r = requests.get(f"{uri}/collections")
assert_http_ok(r)
collections = r.json()["result"]["collections"]
filtered_collections = [c for c in collections if c['name'] == collection_name]
if len(filtered_collections) == 0:
print(
f"Collection '{collection_name}' does not exist on peer {uri} found {json.dumps(collections, indent=4)}")
return False
else:
continue
return True
def check_collection_local_shards_count(peer_api_uri: str, collection_name: str,
expected_local_shard_count: int) -> bool:
return get_collection_local_shards_count(peer_api_uri, collection_name) == expected_local_shard_count
def get_collection_local_shards_count(peer_api_uri: str, collection_name: str) -> int:
collection_cluster_info = get_collection_cluster_info(peer_api_uri, collection_name)
return len(collection_cluster_info["local_shards"])
def check_collection_local_shards_point_count(peer_api_uri: str, collection_name: str,
expected_count: int) -> int:
collection_cluster_info = get_collection_cluster_info(peer_api_uri, collection_name)
point_count = sum(map(lambda shard: shard["points_count"], collection_cluster_info["local_shards"]))
is_correct = point_count == expected_count
if not is_correct:
print(f"Collection '{collection_name}' on peer {peer_api_uri} ({point_count} != {expected_count}): {json.dumps(collection_cluster_info, indent=4)}")
return is_correct
def check_collection_shard_transfers_count(peer_api_uri: str, collection_name: str,
expected_shard_transfers_count: int, headers={}) -> bool:
collection_cluster_info = get_collection_cluster_info(peer_api_uri, collection_name, headers=headers)
local_shard_count = len(collection_cluster_info["shard_transfers"])
return local_shard_count == expected_shard_transfers_count
def check_collection_resharding_operations_count(peer_api_uri: str, collection_name: str,
expected_resharding_operations_count: int, headers={}) -> bool:
collection_cluster_info = get_collection_cluster_info(peer_api_uri, collection_name, headers=headers)
# TODO(resharding): until resharding release, the resharding operations are not always exposed
# Once we do release, we can remove the zero fallback here
# See: <https://github.com/qdrant/qdrant/pull/4599>
local_resharding_count = len(collection_cluster_info["resharding_operations"]) if "resharding_operations" in collection_cluster_info else 0
return local_resharding_count == expected_resharding_operations_count
def get_collection_resharding_stages(peer_api_uri: str, collection_name: str, headers={}) -> [str]:
"""
Resharding stages applied on this peer, as `migrating_points`,
`read_hash_ring_committed` or `write_hash_ring_committed`.
The collection cluster info hides the stage on purpose, only telemetry
exposes it. Reading it goes through the same shard holder lock that the
consensus handlers take while applying a stage change, so a stage seen here
is fully applied, including any side effects like invalidating shard clean tasks.
"""
r = requests.get(f"{peer_api_uri}/telemetry", params={"details_level": 3}, headers=headers)
assert_http_ok(r)
for collection in r.json()["result"]["collections"]["collections"]:
if collection["id"] == collection_name:
return [operation["stage"] for operation in collection.get("resharding") or []]
return []
def check_collection_resharding_operation_stage(peer_api_uri: str, collection_name: str, expected_stage: str, headers={}) -> bool:
return expected_stage in get_collection_resharding_stages(peer_api_uri, collection_name, headers=headers)
def check_collection_shard_transfer_method(peer_api_uri: str, collection_name: str,
expected_method: str) -> bool:
collection_cluster_info = get_collection_cluster_info(peer_api_uri, collection_name)
# Shortcut if no transfers
if len(collection_cluster_info["shard_transfers"]) == 0:
print(f"check_collection_shard_transfer_method: no transfers for collection '{collection_name}'")
return False
# Check method on each transfer
for transfer in collection_cluster_info["shard_transfers"]:
if "method" not in transfer:
print(f"check_collection_shard_transfer_method: unknown method not match expected '{expected_method}'")
continue
method = transfer["method"]
if method == expected_method:
return True
else:
print(f"check_collection_shard_transfer_method: method '{method}' does not match expected '{expected_method}'")
return False
def check_collection_shard_transfer_progress(peer_api_uri: str, collection_name: str,
expected_transfer_progress: int,
expected_transfer_total: int) -> bool:
collection_cluster_info = get_collection_cluster_info(peer_api_uri, collection_name)
# Check progress on each transfer
for transfer in collection_cluster_info["shard_transfers"]:
if "comment" not in transfer:
continue
comment = transfer["comment"]
# Compare progress or total
m = re.search(r"Transferring records \((\d+)/(\d+)\)", comment)
if m is None:
continue
current, total = m.groups()
if current is not None and expected_transfer_progress is not None and int(
current) >= expected_transfer_progress:
return True
if total is not None and expected_transfer_total is not None and int(total) >= expected_transfer_total:
return True
return False
def check_all_replicas_active(peer_api_uri: str, collection_name: str, headers={}, min_local_replicas=0) -> bool:
try:
collection_cluster_info = get_collection_cluster_info(peer_api_uri, collection_name, headers=headers)
if len(collection_cluster_info["local_shards"]) < min_local_replicas:
return False
for shard in collection_cluster_info["local_shards"]:
if shard['state'] != 'Active':
return False
for shard in collection_cluster_info["remote_shards"]:
if shard['state'] != 'Active':
return False
except requests.exceptions.ConnectionError:
return False
return True
def check_some_replicas_not_active(peer_api_uri: str, collection_name: str) -> bool:
return not check_all_replicas_active(peer_api_uri, collection_name)
def check_collection_cluster(peer_url, collection_name):
res = requests.get(f"{peer_url}/collections/{collection_name}/cluster", timeout=10)
assert_http_ok(res)
local_shards = res.json()["result"]['local_shards']
# During snapshot shard transfer recovery the existing local shard is
# temporarily taken before the snapshot is installed, so `local_shards` may
# be empty for a brief window. Report it as a non-Active state so callers
# poll again instead of crashing with IndexError.
if not local_shards:
return {'state': 'Absent', 'points_count': 0}
return local_shards[0]
def check_strict_mode_enabled(peer_api_uri: str, collection_name: str) -> bool:
collection_info = get_collection_info(peer_api_uri, collection_name)
if "strict_mode_config" not in collection_info["config"]:
return False
strict_mode_enabled = collection_info["config"]["strict_mode_config"]["enabled"]
return strict_mode_enabled == True
def check_strict_mode_disabled(peer_api_uri: str, collection_name: str) -> bool:
collection_info = get_collection_info(peer_api_uri, collection_name)
if "strict_mode_config" not in collection_info["config"]:
return True
strict_mode_enabled = collection_info["config"]["strict_mode_config"]["enabled"]
return strict_mode_enabled == False
def wait_peer_added(peer_api_uri: str, expected_size: int = 1, headers={}) -> str:
wait_for(check_cluster_size, peer_api_uri, expected_size, headers=headers)
wait_for(leader_is_defined, peer_api_uri, headers=headers)
return get_leader(peer_api_uri, headers=headers)
def wait_for_collection(peer_api_uri: str, collection_name: str):
def is_collection_listed() -> bool:
try:
res = requests.get(f"{peer_api_uri}/collections")
if not res.ok:
return False
collections = set(collection['name'] for collection in res.json()["result"]['collections'])
return collection_name in collections
except requests.exceptions.ConnectionError:
return False
wait_for(is_collection_listed)
def wait_collection_green(peer_api_uri: str, collection_name: str):
try:
wait_for(check_collection_green, peer_api_uri, collection_name)
except Exception as e:
print_clusters_info([peer_api_uri])
raise e
def wait_for_some_replicas_not_active(peer_api_uri: str, collection_name: str):
try:
wait_for(check_some_replicas_not_active, peer_api_uri, collection_name)
except Exception as e:
print_clusters_info([peer_api_uri])
raise e
def wait_for_all_replicas_active(peer_api_uri: str, collection_name: str, headers={}, min_local_replicas=0):
try:
wait_for(check_all_replicas_active, peer_api_uri, collection_name, headers=headers, min_local_replicas=min_local_replicas)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name, headers=headers)
raise e
def wait_for_uniform_cluster_status(peer_api_uris: [str], expected_leader: Optional[str] = None, headers={}):
"""Wait for membership size and leader agreement, optionally requiring a specific leader."""
try:
wait_for(all_nodes_cluster_info_consistent, peer_api_uris, expected_leader, headers=headers)
except Exception as e:
print_clusters_info(peer_api_uris)
raise e
def wait_for_all_peers_versions(peer_api_uris: [str]):
try:
wait_for(peers_have_version, peer_api_uris)
except Exception as e:
print_clusters_info(peer_api_uris)
raise e
def wait_for_same_commit(peer_api_uris: [str]):
try:
wait_for(all_nodes_have_same_commit, peer_api_uris)
except Exception as e:
print_clusters_info(peer_api_uris)
raise e
def wait_for_same_applied_commit(peer_api_uris: [str]):
try:
wait_for(all_nodes_have_applied_same_commit, peer_api_uris)
except Exception as e:
print_clusters_info(peer_api_uris)
raise e
def wait_all_peers_up(peer_api_uris: [str]):
try:
wait_for(all_nodes_respond, peer_api_uris)
except Exception as e:
print_clusters_info(peer_api_uris)
raise e
def wait_for_uniform_collection_existence(collection_name: str, peer_api_uris: [str]):
try:
wait_for(collection_exists_on_all_peers, collection_name, peer_api_uris)
except Exception as e:
print_clusters_info(peer_api_uris)
raise e
def wait_for_collection_shard_transfers_count(peer_api_uri: str, collection_name: str,
expected_shard_transfer_count: int, headers={}):
try:
wait_for(check_collection_shard_transfers_count, peer_api_uri, collection_name, expected_shard_transfer_count, headers=headers)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name, headers=headers)
raise e
def wait_for_collection_shard_transfer_method(peer_api_uri: str, collection_name: str,
expected_method: str):
try:
wait_for(check_collection_shard_transfer_method, peer_api_uri, collection_name, expected_method)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name)
raise e
def wait_for_collection_shard_transfer_progress(peer_api_uri: str, collection_name: str,
expected_transfer_progress: int = None,
expected_transfer_total: int = None):
try:
wait_for(check_collection_shard_transfer_progress, peer_api_uri, collection_name, expected_transfer_progress,
expected_transfer_total, wait_for_interval=0.1)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name)
raise e
def wait_for_collection_resharding_operations_count(peer_api_uri: str,
collection_name: str,
expected_resharding_operations_count:
int, headers={}, **kwargs):
try:
wait_for(check_collection_resharding_operations_count, peer_api_uri, collection_name, expected_resharding_operations_count, headers=headers, **kwargs)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name, headers=headers)
raise e
def wait_for_collection_resharding_operation_stage(peer_api_uri: str, collection_name: str, expected_stage: str, headers={}):
try:
wait_for(check_collection_resharding_operation_stage, peer_api_uri, collection_name, expected_stage, headers=headers)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name, headers=headers)
raise e
def wait_for_collection_local_shards_count(peer_api_uri: str, collection_name: str, expected_local_shard_count: int):
try:
wait_for(check_collection_local_shards_count, peer_api_uri, collection_name, expected_local_shard_count)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name)
raise e
def wait_for_strict_mode_enabled(peer_api_uri: str, collection_name: str):
try:
wait_for(check_strict_mode_enabled, peer_api_uri, collection_name)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name)
raise e
def wait_for_strict_mode_disabled(peer_api_uri: str, collection_name: str):
try:
wait_for(check_strict_mode_disabled, peer_api_uri, collection_name)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name)
raise e
def wait_for(condition: Callable[..., bool], *args, wait_for_timeout=WAIT_TIME_SEC, wait_for_interval=RETRY_INTERVAL_SEC, **kwargs):
start = time.monotonic()
while not condition(*args, **kwargs):
elapsed = time.monotonic() - start
if elapsed > wait_for_timeout:
raise Exception(
f"Timeout waiting for condition {condition.__name__} to be satisfied in {wait_for_timeout} seconds")
else:
time.sleep(wait_for_interval)
def peer_is_online(peer_api_uri: str, path: str = "/readyz") -> bool:
try:
r = requests.get(f"{peer_api_uri}{path}", timeout=10)
return r.status_code == 200
except:
return False
def wait_for_peer_online(peer_api_uri: str, path="/readyz", wait_for_timeout=WAIT_TIME_SEC):
try:
wait_for(peer_is_online, peer_api_uri, path=path, wait_for_timeout=wait_for_timeout)
except Exception as e:
print_clusters_info([peer_api_uri])
raise e
def check_collection_green(peer_api_uri: str, collection_name: str, expected_status: str = "green") -> bool:
collection_cluster_info = get_collection_info(peer_api_uri, collection_name)
return collection_cluster_info['status'] == expected_status
def check_collection_points_count(peer_api_uri: str, collection_name: str, expected_size: int) -> bool:
collection_cluster_info = get_collection_info(peer_api_uri, collection_name)
return collection_cluster_info['points_count'] == expected_size
def wait_collection_points_count(peer_api_uri: str, collection_name: str, expected_size: int):
try:
wait_for(check_collection_points_count, peer_api_uri, collection_name, expected_size)
except Exception as e:
print_collection_cluster_info(peer_api_uri, collection_name)
raise e
def wait_collection_on_all_peers(collection_name: str, peer_api_uris: [str], max_wait=30, headers={}):
# Check that it exists on all peers
while True:
exists = True
for url in peer_api_uris:
r = requests.get(f"{url}/collections", headers=headers)
assert_http_ok(r)
collections = r.json()["result"]["collections"]
exists &= any(collection["name"] == collection_name for collection in collections)
if exists:
break
else:
# Wait until collection is created on all peers
# Consensus guarantees that collection will appear on majority of peers, but not on all of them
# So we need to wait a bit extra time
time.sleep(1)
max_wait -= 1
if max_wait <= 0:
raise Exception("Collection was not created on all peers in time")
def wait_collection_exists_and_active_on_all_peers(collection_name: str, peer_api_uris: [str], max_wait=30, headers={}):
wait_collection_on_all_peers(collection_name, peer_api_uris, max_wait, headers=headers)
for peer_uri in peer_api_uris:
# Collection is active on all peers
wait_for_all_replicas_active(collection_name=collection_name, peer_api_uri=peer_uri, headers=headers)
def create_shard_key(
shard_key,
peer_url,
collection="test_collection",
shard_number=None,
replication_factor=None,
placement=None,
timeout=10,
headers={},
):
r_create = requests.put(
f"{peer_url}/collections/{collection}/shards?timeout={timeout}",
json={
"shard_key": shard_key,
"shards_number": shard_number,
"replication_factor": replication_factor,
"placement": placement,
},
headers=headers,
)
assert_http_ok(r_create)
def move_shard(source_uri, collection_name, shard_id, source_peer_id, target_peer_id):
r = requests.post(
f"{source_uri}/collections/{collection_name}/cluster", json={
"move_shard": {
"shard_id": shard_id,
"from_peer_id": source_peer_id,
"to_peer_id": target_peer_id
}
})
assert_http_ok(r)
def replicate_shard(source_uri, collection_name, shard_id, source_peer_id, target_peer_id, method=None):
payload = {
"shard_id": shard_id,
"from_peer_id": source_peer_id,
"to_peer_id": target_peer_id
}
if method:
payload["method"] = method
r = requests.post(
f"{source_uri}/collections/{collection_name}/cluster", json={
"replicate_shard": payload
}, timeout=WAIT_TIME_SEC)
assert_http_ok(r)
def check_data_consistency(data):
assert(len(data) > 1)
for i in range(len(data) - 1):
j = i + 1
data_i = data[i]
data_j = data[j]
if data_i != data_j:
ids_i = set(x.get("id") for x in data_i)
ids_j = set(x.get("id") for x in data_j)
diff = ids_i - ids_j
if len(diff) < 100:
print(f"Diff between {i} and {j}: {diff}")
else:
sample = list(diff)[:32]
print(f"Diff len between {i} and {j}: {len(diff)}, sample: {sample}")
assert False, "Data on all nodes should be consistent"
def check_feature_enabled(peer_api_uri, feature) -> bool:
r = requests.get(f"{peer_api_uri}/telemetry", params={"details_level": 10})
assert_http_ok(r)
features = r.json()['result']['app']['features']
result = features[feature]
return result
def skip_if_no_feature(peer_api_uri, feature):
feature_is_enabled = check_feature_enabled(peer_api_uri, feature)
if not feature_is_enabled:
pytest.skip(f"Skipping because the feature {feature} is disabled at runtime.")