Skip to content

Commit 00091a4

Browse files
committed
fix tests
Signed-off-by: Alex Chi Z <[email protected]>
1 parent d37cd4b commit 00091a4

File tree

3 files changed

+12
-2
lines changed

3 files changed

+12
-2
lines changed

test_runner/regress/test_attach_tenant_config.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,6 @@ def test_fully_custom_config(positive_env: NeonEnv):
194194
"numerator": 0,
195195
"denominator": 10,
196196
},
197-
"rel_size_v1_access_disabled": True,
198197
}
199198

200199
vps_http = env.storage_controller.pageserver_api()

test_runner/regress/test_compaction.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,12 @@
1212
from fixtures.neon_fixtures import (
1313
NeonEnvBuilder,
1414
generate_uploads_and_deletions,
15+
wait_for_last_flush_lsn,
1516
)
1617
from fixtures.pageserver.http import PageserverApiException
1718
from fixtures.utils import skip_in_debug_build, wait_until
1819
from fixtures.workload import Workload
20+
from regress.test_pg_regress import patch_tenant_conf
1921

2022
AGGRESSIVE_COMPACTION_TENANT_CONF = {
2123
# Disable gc and compaction. The test runs compaction manually.
@@ -1101,7 +1103,8 @@ def test_image_consistent_lsn(neon_env_builder: NeonEnvBuilder):
11011103
neon_env_builder.num_pageservers = 2
11021104
neon_env_builder.num_safekeepers = 1
11031105
env = neon_env_builder.init_start(
1104-
initial_tenant_conf=tenant_conf,
1106+
# We have to run this test with v1 to avoid v2 migration producing too many delta layers.
1107+
initial_tenant_conf=patch_tenant_conf(tenant_conf, "v1"),
11051108
initial_tenant_shard_count=4,
11061109
initial_tenant_shard_stripe_size=1,
11071110
)
@@ -1116,6 +1119,8 @@ def test_image_consistent_lsn(neon_env_builder: NeonEnvBuilder):
11161119
f"INSERT INTO foo (id, val) VALUES ({v}, repeat('abcde{v:0>3}', 500))", log_query=False
11171120
)
11181121

1122+
wait_for_last_flush_lsn(env, endpoint, env.initial_tenant, env.initial_timeline)
1123+
11191124
response = env.storage_controller.tenant_timeline_describe(tenant_id, timeline_id)
11201125
shards = response["shards"]
11211126
for shard in shards:

test_runner/regress/test_compatibility.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -585,6 +585,12 @@ def test_historic_storage_formats(
585585
dataset.tenant_id,
586586
{"rel_size_v1_access_disabled": True, "rel_size_v2_enabled": True},
587587
)
588+
for timeline in timelines:
589+
env.pageserver.http_client().timeline_patch_index_part(
590+
dataset.tenant_id,
591+
timeline["timeline_id"],
592+
{"force_index_update": True, "rel_size_migration": "migrated"},
593+
)
588594

589595
# Import tenant does not create the timeline on safekeepers,
590596
# because it is a debug handler and the timeline may have already been

0 commit comments

Comments
 (0)