diff mbox series

[1/5] lttng-tools: ensure that ptest errors are not suppressed

Message ID 20260216133351.1034740-1-alex.kanavin@gmail.com
State Under Review
Headers show
Series [1/5] lttng-tools: ensure that ptest errors are not suppressed | expand

Commit Message

Alexander Kanavin Feb. 16, 2026, 1:33 p.m. UTC 
From: Alexander Kanavin <alex@linutronix.de>

At some recent point lttng-tools ptests have quietly regressed,
and most of them aren't executed anymore. Errors are printed
on the console, but aren't reported as exit code from run-ptest.

The reason is that exitcode was set to what sed returned, not make.
The original reason for piping through sed was to unify /tmp/tmp.xxxx outputs
for easier results comparison, but (after fixing the tests) I don't
see such lines anymore, and in any case ensuring such regressions
are caught is more important.

With this fix, run-ptest and testimage starts to fail as it should.

Signed-off-by: Alexander Kanavin <alex@linutronix.de>
---
 meta/recipes-kernel/lttng/lttng-tools/run-ptest | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/meta/recipes-kernel/lttng/lttng-tools/run-ptest b/meta/recipes-kernel/lttng/lttng-tools/run-ptest
index f4f7a3baea..e7edc03e96 100755
--- a/meta/recipes-kernel/lttng/lttng-tools/run-ptest
+++ b/meta/recipes-kernel/lttng/lttng-tools/run-ptest
@@ -31,7 +31,7 @@  validate_lttng_modules_present || {
 make -k -t all >error.log 2>&1
 # Can specify a test e.g.:
 # -C tests/regression/ check TESTS='kernel/test_callstack'
-make -k -s $makeargs check 2>error.log | sed -e 's#/tmp/tmp\...........#/tmp/tmp.XXXXXXXXXX#g'
+make -k -s $makeargs check 2>error.log
 exitcode=$?
 if [ -e error.log ]; then
     cat error.log