diff mbox series

[8/8] oeqa/runtime/parselogs: select the correct machine-specific ignores early

Message ID 20230921134833.582827-9-ross.burton@arm.com
State Accepted, archived
Commit 124dede2b20930d09a5cc319cb9333c5a912fa48
Headers show
Series oeqa/runtime/parselogs improvements | expand

Commit Message

Ross Burton Sept. 21, 2023, 1:48 p.m. UTC 
From: Ross Burton <ross.burton@arm.com>

This has no impact to the execution, but makes the following changes
neater.

Signed-off-by: Ross Burton <ross.burton@arm.com>
---
 meta/lib/oeqa/runtime/cases/parselogs.py | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)
diff mbox series

Patch

diff --git a/meta/lib/oeqa/runtime/cases/parselogs.py b/meta/lib/oeqa/runtime/cases/parselogs.py
index 3f205661ea9..c889be8f6ea 100644
--- a/meta/lib/oeqa/runtime/cases/parselogs.py
+++ b/meta/lib/oeqa/runtime/cases/parselogs.py
@@ -218,7 +218,11 @@  class ParseLogsTest(OERuntimeTestCase):
                 'Ordering cycle found, skipping',
                 ])
 
-        cls.ignore_errors = ignore_errors
+        try:
+            cls.ignore_errors = ignore_errors[cls.td.get('MACHINE')]
+        except KeyError:
+            cls.logger.info('No ignore list found for this machine, using default')
+            cls.ignore_errors = cls.ignore_errors['default']
 
     # Go through the log locations provided and if it's a folder
     # create a list with all the .log files in it, if it's a file
@@ -268,13 +272,8 @@  class ParseLogsTest(OERuntimeTestCase):
         grepcmd = grepcmd[:-1]
         grepcmd += '" ' + str(log) + " | grep -Eiv \'"
 
-        try:
-            errorlist = self.ignore_errors[self.td.get('MACHINE')]
-        except KeyError:
-            self.msg += 'No ignore list found for this machine, using default\n'
-            errorlist = self.ignore_errors['default']
 
-        for ignore_error in errorlist:
+        for ignore_error in self.ignore_errors:
             ignore_error = ignore_error.replace('(', r'\(')
             ignore_error = ignore_error.replace(')', r'\)')
             ignore_error = ignore_error.replace("'", '.')