Skip to content

Commit e9c8a60

Browse files
committed
fix(log): ignore serialization format error globally
Move the "expected change in the serialization format" error from a test-specific ignore rule to the global ERRORS_IGNORED list in logfiles.py. This ensures the error is ignored consistently across all tests, simplifying maintenance and reducing duplication.
1 parent 06b9bae commit e9c8a60

File tree

2 files changed

+1
-7
lines changed

2 files changed

+1
-7
lines changed

cardano_node_tests/tests/test_node_upgrade.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -93,13 +93,6 @@ def test_ignore_log_errors(
9393
"""Ignore selected errors in log right after node upgrade."""
9494
common.get_test_id(cluster_singleton)
9595

96-
logfiles.add_ignore_rule(
97-
files_glob="*.stdout",
98-
regex="ChainDB:Error:.* Invalid snapshot DiskSnapshot .*DeserialiseFailure "
99-
".* expected change in the serialization format",
100-
ignore_file_id=worker_id,
101-
)
102-
10396
# The error should be present only when upgrading pre UTxO-HD release.
10497
# The UTxO-HD was added in 10.4.1, so when we are upgrading from 10.4.1+ release to
10598
# 10.5.0+ release, the error should not be there.

cardano_node_tests/utils/logfiles.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@
5959
"DiffusionError thread killed",
6060
# TODO: connection to other node before the other node is started
6161
r"AcquireConnectionError Network\.Socket\.connect",
62+
"expected change in the serialization format",
6263
]
6364
# Already removed from the list above:
6465
# * Workaround for node issue https://github.com/IntersectMBO/cardano-node/issues/4369

0 commit comments

Comments
 (0)