From: Martin Klozik <martin.klozik@tieto.com>
Date: Thu, 28 Jun 2018 13:30:47 +0000 (+0200)
Subject: ci: Support for negative pylint rating
X-Git-Tag: opnfv-7.0.0~10
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F03%2F59203%2F1;p=vswitchperf.git

ci: Support for negative pylint rating

Script ./check was updated to correctly parse and process negative pylint
ratings. Also awk expression was fixed to get rid of error messages
during parsing of pylint output.

JIRA: VSPERF-587

Change-Id: I72c5bd85f38274f308fce881517e5b6fa4a123dc
Signed-off-by: Martin Klozik <martin.klozik@tieto.com>
Reviewed-by: Al Morton <acmorton@att.com>
Reviewed-by: Christian Trautman <ctrautma@redhat.com>
Reviewed-by: Sridhar Rao <sridhar.rao@spirent.com>
Reviewed-by: Richard Elias <richard.elias@tieto.com>
---

diff --git a/check b/check
index 3c8c2927..b1c79ac4 100755
--- a/check
+++ b/check
@@ -159,7 +159,7 @@ if [ -s $FILE_LIST ] ; then
         fi
         # run pylint and extract final rating
         output=`$PYLINT --rcfile $PYLINT_RC $pyfile 2>/dev/null`
-        rating=`echo -e $output | tail -n3 | grep rated | sed -e 's/^.*rated at \([0-9.]*\).*$/\1/'`
+        rating=`echo -e $output | tail -n3 | grep rated | sed -e 's/^.*rated at \(-\?[0-9.]*\).*$/\1/'`
         # evaluate and display aquired rating
         if [ "x$rating" == "x" ] ; then
             # rating is not available for files without python statements
@@ -167,7 +167,7 @@ if [ -s $FILE_LIST ] ; then
         elif rating_is_ok $rating ; then
             printf "    %-70s ${GREEN}%-6s${BLACK}\n" $pyfile "OK"
         else
-            echo -e "$output" | awk '/^*+ Module|^[A-Z]\:/'
+            echo -e "$output" | awk '/^\*+ Module|^[A-Z]\:/'
             printf "    %-70s ${RED}%-6s${BLACK}\n" $pyfile $rating
         fi
     done