From a582977bbc22bbe63e7239368791e464e51b750b Mon Sep 17 00:00:00 2001 From: Victor Kamensky Date: Mon, 16 Jun 2025 22:25:40 -0700 Subject: [PATCH] oeqa/runtime/stap: fix script logic problem Looks as a typo introduced by "6cf4d23a2d26c2767edd93f2eb317ff759b5a992 (oeqa/runtime/stap: improve systemtap test)" - when original command was split into two parts 'self.target.run' was missed for the second part. Note this and previos commit issues were masked because mentioned commit has try/except around tests, so failures that normally are reported when test assertEqual throw an exception never reached oeqa infra. Effectively, this test was broken for the last 3 years. I will leave blanket try/except around test for now, so this test will pass all the time as long as dmesg is working. But at least with the fixes one can inspect log.do_testimage output and check that all steps are successful or not. (From OE-Core rev: ec6e4fa83b8d64035b61c080402151af04b3d137) Signed-off-by: Victor Kamensky Signed-off-by: Richard Purdie --- meta/lib/oeqa/runtime/cases/stap.py | 1 + 1 file changed, 1 insertion(+) diff --git a/meta/lib/oeqa/runtime/cases/stap.py b/meta/lib/oeqa/runtime/cases/stap.py index 23eace7e9d..6b55e7de50 100644 --- a/meta/lib/oeqa/runtime/cases/stap.py +++ b/meta/lib/oeqa/runtime/cases/stap.py @@ -26,6 +26,7 @@ class StapTest(OERuntimeTestCase): self.assertEqual(status, 0, msg='\n'.join([cmd, output])) cmd = 'staprun -v -R -b1 stap_hello.ko' + status, output = self.target.run(cmd, 60) self.assertEqual(status, 0, msg='\n'.join([cmd, output])) self.assertIn('Hello, SystemTap!', output, msg='\n'.join([cmd, output])) except: