mirror of
https://github.com/neondatabase/neon.git
synced 2026-01-15 01:12:56 +00:00
tests: allow list breaching min resident size in statvfs test (#8358)
## Problem This test would sometimes violate the min resident size during disk eviction and fail due to the generate warning log. Disk usage candidate collection only takes into account active tenants. However, the statvfs call takes into account the entire tenants directory, which includes tenants which haven't become active yet. After re-starting the pageserver, disk usage eviction may kick in *before* both tenants have become active. Hence, the logic will try to satisfy thedisk usage requirements by evicting everything belonging to the active tenant, and hence violating the tenant minimum resident size. ## Summary of changes Allow the warning
This commit is contained in:
@@ -794,6 +794,16 @@ def test_statvfs_pressure_usage(eviction_env: EvictionEnv):
|
||||
|
||||
wait_until(2, 2, less_than_max_usage_pct)
|
||||
|
||||
# Disk usage candidate collection only takes into account active tenants.
|
||||
# However, the statvfs call takes into account the entire tenants directory,
|
||||
# which includes tenants which haven't become active yet.
|
||||
#
|
||||
# After re-starting the pageserver, disk usage eviction may kick in *before*
|
||||
# both tenants have become active. Hence, the logic will try to satisfy the
|
||||
# disk usage requirements by evicting everything belonging to the active tenant,
|
||||
# and hence violating the tenant minimum resident size.
|
||||
env.neon_env.pageserver.allowed_errors.append(".*" + GLOBAL_LRU_LOG_LINE)
|
||||
|
||||
|
||||
def test_statvfs_pressure_min_avail_bytes(eviction_env: EvictionEnv):
|
||||
"""
|
||||
|
||||
Reference in New Issue
Block a user