From 5b47a5126fa35a5736e180dfdc00f2f83687797f Mon Sep 17 00:00:00 2001 From: Christian Schwarz Date: Wed, 28 Feb 2024 10:25:28 +0100 Subject: [PATCH] unbreak `test_forward_compatibility`, broken by reverts It's failing because the previous version includes the layer flushing we are trying to revert, but we are removing the allowed log line from the test code ([here](https://github.com/neondatabase/neon/pull/6938/files#diff-a89c9a474d3d87fc497177cdb54142579b53fce222da282f233027555d972128L89)). --- test_runner/fixtures/pageserver/allowed_errors.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/test_runner/fixtures/pageserver/allowed_errors.py b/test_runner/fixtures/pageserver/allowed_errors.py index 74c6bddf23..8ff4341cc0 100755 --- a/test_runner/fixtures/pageserver/allowed_errors.py +++ b/test_runner/fixtures/pageserver/allowed_errors.py @@ -82,6 +82,11 @@ DEFAULT_PAGESERVER_ALLOWED_ERRORS = ( # During shutdown, DownloadError::Cancelled may be logged as an error. Cleaning this # up is tracked in https://github.com/neondatabase/neon/issues/6096 ".*Cancelled, shutting down.*", + # Open layers are only rolled at Lsn boundaries to avoid name clashses. + # Hence, we can overshoot the soft limit set by checkpoint distance. + # This is especially pronounced in tests that set small checkpoint + # distances. + ".*Flushed oversized open layer with size.*", )