[PATCH] oeqa/parselogs: Updated whitelist

[PATCH] oeqa/parselogs: Updated whitelist

[Lucian Musat]

Moved an error to common, and whitelisted the BAR errors
and ati module. Also the uvesafb related errors will be
ignored in LSB images which do not have graphic interface.

Signed-off-by: Lucian Musat <george.l.musat@intel.com>
---
 meta/lib/oeqa/runtime/parselogs.py | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/meta/lib/oeqa/runtime/parselogs.py b/meta/lib/oeqa/runtime/parselogs.py
index fc2bc38..114f0bf 100644
--- a/meta/lib/oeqa/runtime/parselogs.py
+++ b/meta/lib/oeqa/runtime/parselogs.py
@@ -38,8 +38,15 @@ common_errors = [
     'Online check failed for',
     'netlink init failed',
     'Fast TSC calibration',
+    "BAR 0-9",
+    "Failed to load module \"ati\"",
+    "controller can't do DEVSLP, turning off",
     ]
 
+video_related = [
+    "uvesafb",
+]
+
 x86_common = [
     '[drm:psb_do_init] *ERROR* Debug is',
     'wrong ELF class',
@@ -99,11 +106,7 @@ ignore_errors = {
         '(EE) Failed to load module psbdrv',
         '(EE) open /dev/fb0: No such file or directory',
         '(EE) AIGLX: reverting to software rendering',
-        "controller can't do DEVSLP, turning off",
         ] + x86_common,
-    'intel-corei7-64' : [
-        "controller can't do DEVSLP, turning off",
-        ] + common_errors,
     'crownbay' : x86_common,
     'genericx86' : x86_common,
     'genericx86-64' : x86_common,
@@ -126,6 +129,10 @@ class ParseLogsTest(oeRuntimeTest):
         self.ignore_errors = ignore_errors
         self.log_locations = log_locations
         self.msg = ""
+        (is_lsb, location) = oeRuntimeTest.tc.target.run("which LSB_Test.sh")
+        if is_lsb == 0:
+            for machine in self.ignore_errors:
+                self.ignore_errors[machine] = self.ignore_errors[machine] + video_related
 
     def getMachine(self):
         return oeRuntimeTest.tc.d.getVar("MACHINE", True)
@@ -200,6 +207,7 @@ class ParseLogsTest(oeRuntimeTest):
             ignore_error = ignore_error.replace("[", "\[")
             ignore_error = ignore_error.replace("]", "\]")
             ignore_error = ignore_error.replace("*", "\*")
+            ignore_error = ignore_error.replace("0-9", "[0-9]")
             grepcmd += ignore_error+"|"
         grepcmd = grepcmd[:-1]
         grepcmd += "\'"
-- 
2.5.0

[PATCH] oeqa/parselogs: Updated whitelist

[Cristina Agurida]

Moved an error to common, and whitelisted the BAR errors
and ati module. Also the uvesafb related errors will be
ignored in LSB images which do not have graphic interface.

Fix for [YOCTO #8387]
Fix for [YOCTO #8590]
Fix for [YOCTO #6820]
Fix for [YOCTO #7897]

Signed-off-by: Lucian Musat <george.l.musat at intel.com>
Signed-off-by: Cristina Agurida <cristina-danielax.agurida@intel.com>
---
 meta/lib/oeqa/runtime/parselogs.py | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/meta/lib/oeqa/runtime/parselogs.py b/meta/lib/oeqa/runtime/parselogs.py
index 5a2d50f..44834ce 100644
--- a/meta/lib/oeqa/runtime/parselogs.py
+++ b/meta/lib/oeqa/runtime/parselogs.py
@@ -38,8 +38,17 @@ common_errors = [
     'Online check failed for',
     'netlink init failed',
     'Fast TSC calibration',
+    "BAR 0-9",
+    "Failed to load module \"ati\"",
+    "controller can't do DEVSLP, turning off",
+    "stmmac_dvr_probe: warning: cannot get CSR clock",
+    "error: couldn\'t mount because of unsupported optional features",
     ]
 
+video_related = [
+    "uvesafb",
+]
+
 x86_common = [
     '[drm:psb_do_init] *ERROR* Debug is',
     'wrong ELF class',
@@ -99,11 +108,7 @@ ignore_errors = {
         '(EE) Failed to load module psbdrv',
         '(EE) open /dev/fb0: No such file or directory',
         '(EE) AIGLX: reverting to software rendering',
-        "controller can't do DEVSLP, turning off",
         ] + x86_common,
-    'intel-corei7-64' : [
-        "controller can't do DEVSLP, turning off",
-        ] + common_errors,
     'crownbay' : x86_common,
     'genericx86' : x86_common,
     'genericx86-64' : x86_common,
@@ -126,6 +131,10 @@ class ParseLogsTest(oeRuntimeTest):
         self.ignore_errors = ignore_errors
         self.log_locations = log_locations
         self.msg = ""
+        (is_lsb, location) = oeRuntimeTest.tc.target.run("which LSB_Test.sh")
+        if is_lsb == 0:
+            for machine in self.ignore_errors:
+                self.ignore_errors[machine] = self.ignore_errors[machine] + video_related
 
     def getMachine(self):
         return oeRuntimeTest.tc.d.getVar("MACHINE", True)
@@ -200,6 +209,7 @@ class ParseLogsTest(oeRuntimeTest):
             ignore_error = ignore_error.replace("[", "\[")
             ignore_error = ignore_error.replace("]", "\]")
             ignore_error = ignore_error.replace("*", "\*")
+            ignore_error = ignore_error.replace("0-9", "[0-9]")
             grepcmd += ignore_error+"|"
         grepcmd = grepcmd[:-1]
         grepcmd += "\'"
-- 
2.1.4

Re: [PATCH] oeqa/parselogs: Updated whitelist

[Saul Wold]

Ping, this does not appear to be in -next or ross/mut, any issues?

Sau!

On Thu, 2016-02-04 at 14:13 +0200, Cristina Agurida wrote:
> Moved an error to common, and whitelisted the BAR errors
> and ati module. Also the uvesafb related errors will be
> ignored in LSB images which do not have graphic interface.
> 
> Fix for [YOCTO #8387]
> Fix for [YOCTO #8590]
> Fix for [YOCTO #6820]
> Fix for [YOCTO #7897]
> 
> Signed-off-by: Lucian Musat <george.l.musat at intel.com>
> Signed-off-by: Cristina Agurida <cristina-danielax.agurida@intel.com>
> ---
>  meta/lib/oeqa/runtime/parselogs.py | 18 ++++++++++++++----
>  1 file changed, 14 insertions(+), 4 deletions(-)
> 
> diff --git a/meta/lib/oeqa/runtime/parselogs.py
> b/meta/lib/oeqa/runtime/parselogs.py
> index 5a2d50f..44834ce 100644
> --- a/meta/lib/oeqa/runtime/parselogs.py
> +++ b/meta/lib/oeqa/runtime/parselogs.py
> @@ -38,8 +38,17 @@ common_errors = [
>      'Online check failed for',
>      'netlink init failed',
>      'Fast TSC calibration',
> +    "BAR 0-9",
> +    "Failed to load module \"ati\"",
> +    "controller can't do DEVSLP, turning off",
> +    "stmmac_dvr_probe: warning: cannot get CSR clock",
> +    "error: couldn\'t mount because of unsupported optional
> features",
>      ]
>  
> +video_related = [
> +    "uvesafb",
> +]
> +
>  x86_common = [
>      '[drm:psb_do_init] *ERROR* Debug is',
>      'wrong ELF class',
> @@ -99,11 +108,7 @@ ignore_errors = {
>          '(EE) Failed to load module psbdrv',
>          '(EE) open /dev/fb0: No such file or directory',
>          '(EE) AIGLX: reverting to software rendering',
> -        "controller can't do DEVSLP, turning off",
>          ] + x86_common,
> -    'intel-corei7-64' : [
> -        "controller can't do DEVSLP, turning off",
> -        ] + common_errors,
>      'crownbay' : x86_common,
>      'genericx86' : x86_common,
>      'genericx86-64' : x86_common,
> @@ -126,6 +131,10 @@ class ParseLogsTest(oeRuntimeTest):
>          self.ignore_errors = ignore_errors
>          self.log_locations = log_locations
>          self.msg = ""
> +        (is_lsb, location) = oeRuntimeTest.tc.target.run("which
> LSB_Test.sh")
> +        if is_lsb == 0:
> +            for machine in self.ignore_errors:
> +                self.ignore_errors[machine] =
> self.ignore_errors[machine] + video_related
>  
>      def getMachine(self):
>          return oeRuntimeTest.tc.d.getVar("MACHINE", True)
> @@ -200,6 +209,7 @@ class ParseLogsTest(oeRuntimeTest):
>              ignore_error = ignore_error.replace("[", "\[")
>              ignore_error = ignore_error.replace("]", "\]")
>              ignore_error = ignore_error.replace("*", "\*")
> +            ignore_error = ignore_error.replace("0-9", "[0-9]")
>              grepcmd += ignore_error+"|"
>          grepcmd = grepcmd[:-1]
>          grepcmd += "\'"
> -- 
> 2.1.4
> 

Re: [PATCH] oeqa/parselogs: Updated whitelist

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 285 bytes --]

On 22 February 2016 at 23:29, Saul Wold <saul.wold@intel.com> wrote:

> Ping, this does not appear to be in -next or ross/mut, any issues?
>

I meant to reply about how the grep code is insanely overcomplicated, but
that can be fixed later.  Marked to merge to mut now.

Ross

[-- Attachment #2: Type: text/html, Size: 691 bytes --]