rust/neon
1
0
Fork 0
mirror of https://github.com/neondatabase/neon.git synced 2026-01-04 12:02:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

test_runner: ignore .___temp files in evict_random_layers (#11509)

Browse Source 
## Problem

`test_location_conf_churn` often fails with `neither image nor delta
layer`, but doesn't say what the file actually is. However, past local
failures have indicated that it might be `.___temp` files.

Touches https://github.com/neondatabase/neon/issues/11348.

## Summary of changes

Ignore `.___temp` files when evicting local layers, and include the file
name in the error message.
This commit is contained in:
Erik Grinaker
2025-04-09 21:03:49 +02:00
committed by GitHub
parent 2c21a65b0b
commit 63ee8e2181
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
test_runner/fixtures/pageserver/common_types.py
View File

@@ -105,7 +105,7 @@ def parse_layer_file_name(file_name: str) -> LayerName:
except InvalidFileName:
pass
raise InvalidFileName("neither image nor delta layer")
raise InvalidFileName(f"neither image nor delta layer: {file_name}")
def is_future_layer(layer_file_name: LayerName, disk_consistent_lsn: Lsn):

2
test_runner/regress/test_pageserver_secondary.py
View File

@@ -61,7 +61,7 @@ def evict_random_layers(
)
client = pageserver.http_client()
for layer in initial_local_layers:
if "ephemeral" in layer.name or "temp_download" in layer.name:
if "ephemeral" in layer.name or "temp_download" in layer.name or ".___temp" in layer.name:
continue
layer_name = parse_layer_file_name(layer.name)