diff mbox series

[v2] patchtest: add stronger indication for failed tests

Message ID 20240216211021.17863-1-simone.p.weiss@posteo.com
State Accepted, archived
Commit 661c0a8fd8fe7bff61cea82778c25cf24d791267
Headers show
Series [v2] patchtest: add stronger indication for failed tests | expand

Commit Message

Simone Weiß Feb. 16, 2024, 9:10 p.m. UTC 
From: Simone Weiß <simone.p.weiss@posteo.com>

Do not only log that there has been an issue but add WARNING before for local
runs. Hopefully this helps to avoid that people reading the log to quickly miss
issues.

Fixes [YOCTO #15389]

Signed-off-by: Simone Weiß <simone.p.weiss@posteo.com>
---
v2:
Fix commit message.

 scripts/patchtest | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/scripts/patchtest b/scripts/patchtest
index d0dc6e8fac..3163420220 100755
--- a/scripts/patchtest
+++ b/scripts/patchtest
@@ -164,9 +164,9 @@  def run(patch, logfile=None):
     if premerge_result == 2 and postmerge_result == 2:
         logger.error('patchtest: No test cases found - did you specify the correct suite directory?')
     if premerge_result == 1 or postmerge_result == 1:
-        logger.error('patchtest: At least one patchtest caused a failure or an error - please check')
+        logger.error('WARNING: patchtest: At least one patchtest caused a failure or an error - please check')
     else:
-        logger.error('patchtest: All patchtests passed')
+        logger.error('OK: patchtest: All patchtests passed')
     print('----------------------------------------------------------------------\n')
     return premerge_result or postmerge_result