From 668880cc30b3147a069bf0dccd1d8bf2f41d55d2 Mon Sep 17 00:00:00 2001 From: Frantisek Sumsal Date: Wed, 24 Jan 2024 10:26:03 +0100 Subject: [PATCH] test: don't store udev worker coredumps in journal udev before #30532 may kill the worker process together with a slow program, and when running with sanitizers the resulting coredump might be too big to fit into journal (or the space currently available for journal): [ 30.086194] systemd-journald[330]: Failed to write entry to /var/log/journal/e87de9ccbacf4b88924ff6d9ecaaa82d/system.journal (50 items, 68326399 bytes) despite vacuuming, ignoring: Argument list too long This then makes the test fail, as it checks for the presence of the coredump. Since we don't really care about the coredump in this specific case (as it is an expected one), let's just temporarily override the testsuite-wide Storage=journal and store the coredumps externally. This is a v255-stable-only patch, since after #30532 the test no longer checks for coredumps. (cherry picked from commit 27fc7a03218464ae9b3395120574b4c06e4c8322) --- test/units/testsuite-17.03.sh | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/test/units/testsuite-17.03.sh b/test/units/testsuite-17.03.sh index 9af706fae1..56e352e1a0 100755 --- a/test/units/testsuite-17.03.sh +++ b/test/units/testsuite-17.03.sh @@ -8,6 +8,9 @@ KILL_PID= setup() { mkdir -p "${TEST_RULE%/*}" [[ -e /etc/udev/udev.conf ]] && cp -f /etc/udev/udev.conf /etc/udev/udev.conf.bak + # Don't bother storing the coredumps in journal for this particular test + mkdir -p /run/systemd/coredump.conf.d/ + echo -ne "[Coredump]\nStorage=external\n" >/run/systemd/coredump.conf.d/99-storage-journal.conf cat >"${TEST_RULE}" <