From c4ce5dda5d14911a0aaaf939ecbcc17c7107a69b Mon Sep 17 00:00:00 2001 From: Vlad Lazar Date: Mon, 10 Feb 2025 23:35:10 +0100 Subject: [PATCH] tests: enable sampling with zero ration in tput tests --- test_runner/fixtures/pageserver/allowed_errors.py | 1 + ...pageserver_max_throughput_getpage_at_latest_lsn.py | 11 +++++++++++ 2 files changed, 12 insertions(+) diff --git a/test_runner/fixtures/pageserver/allowed_errors.py b/test_runner/fixtures/pageserver/allowed_errors.py index abddfa2768..f75ab0b34a 100755 --- a/test_runner/fixtures/pageserver/allowed_errors.py +++ b/test_runner/fixtures/pageserver/allowed_errors.py @@ -106,6 +106,7 @@ DEFAULT_PAGESERVER_ALLOWED_ERRORS = ( ".*delaying layer flush by \\S+ for compaction backpressure.*", ".*stalling layer flushes for compaction backpressure.*", ".*layer roll waiting for flush due to compaction backpressure.*", + ".*BatchSpanProcessor.*", ) diff --git a/test_runner/performance/pageserver/pagebench/test_pageserver_max_throughput_getpage_at_latest_lsn.py b/test_runner/performance/pageserver/pagebench/test_pageserver_max_throughput_getpage_at_latest_lsn.py index 3dbbb197f4..2afc5201f9 100644 --- a/test_runner/performance/pageserver/pagebench/test_pageserver_max_throughput_getpage_at_latest_lsn.py +++ b/test_runner/performance/pageserver/pagebench/test_pageserver_max_throughput_getpage_at_latest_lsn.py @@ -10,6 +10,7 @@ from fixtures.log_helper import log from fixtures.neon_fixtures import ( NeonEnv, NeonEnvBuilder, + PageserverTracingConfig, PgBin, wait_for_last_flush_lsn, ) @@ -113,6 +114,15 @@ def setup_and_run_pagebench_benchmark( neon_env_builder.pageserver_config_override = ( f"page_cache_size={page_cache_size}; max_file_descriptors={max_file_descriptors}" ) + + tracing_config = PageserverTracingConfig( + sampling_ratio=(0, 1000), + endpoint="http://localhost:4318/v1/traces", + protocol="http-binary", + timeout="10s", + ) + neon_env_builder.pageserver_tracing_config = tracing_config + ratio = tracing_config.sampling_ratio[0] / tracing_config.sampling_ratio[1] params.update( { "pageserver_config_override.page_cache_size": ( @@ -120,6 +130,7 @@ def setup_and_run_pagebench_benchmark( {"unit": "byte"}, ), "pageserver_config_override.max_file_descriptors": (max_file_descriptors, {"unit": ""}), + "pageserver_config_override.sampling_ratio": (ratio, {"unit": ""}), } )