From a5a9e426cf17f49f1b0a1b5e495ad2d4c6e8fad9 Mon Sep 17 00:00:00 2001 From: Christian Schwarz Date: Thu, 17 Jul 2025 14:52:55 +0000 Subject: [PATCH] Revert "make test_hot_standby_gc pass by enabling leases" This reverts commit bd06bc54b6f2b7dbc53e604b80a7804ead4cf27d. --- test_runner/regress/test_hot_standby.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/test_runner/regress/test_hot_standby.py b/test_runner/regress/test_hot_standby.py index 552cb0a4d7..1ff61ce8dc 100644 --- a/test_runner/regress/test_hot_standby.py +++ b/test_runner/regress/test_hot_standby.py @@ -126,15 +126,13 @@ def test_2_replicas_start(neon_simple_env: NeonEnv): # # When pause_apply is True we model standby lagging behind primary (e.g. due to # high max_standby_streaming_delay). To prevent pageserver from removing data -# still needed by the standby, the standby compute_ctl maintains a lease -# at apply_lsn. +# still needed by the standby apply LSN is propagated in standby -> safekeepers +# -> broker -> pageserver flow so that pageserver could hold off gc for it. @pytest.mark.parametrize("pause_apply", [False, True]) def test_hot_standby_gc(neon_env_builder: NeonEnvBuilder, pause_apply: bool): tenant_conf = { # set PITR interval to be small, so we can do GC "pitr_interval": "0 s", - # secondaries use leases, override test suite default of "0s" - "lsn_lease_length": "5s", } env = neon_env_builder.init_start(initial_tenant_conf=tenant_conf) timeline_id = env.initial_timeline