Enforce Style Guidelines

Update to fail verify if pep8 compliance not met.
Revert back to Coverage 4.1 to show code coverage
report in Jenkins again

Change-Id: Icb467451f101942255c7944ad796497609dcb07e
Signed-off-by: Mark Beierl <mark.beierl@emc.com>

2 files changed, 11 insertions, 3 deletions

diff --git a/ci/setup.py b/ci/setup.py
index a293e9c..2a02276 100755
--- a/ci/setup.py
+++ b/ci/setup.py
@@ -25,7 +25,6 @@ setup(
                       "flask-restful==0.3.5",
                       "flask-restful-swagger==0.19",
                       "html2text==2016.1.8",
-                      "matplotlib==1.3.1",
                       "python-cinderclient==1.6.0",
                       "python-glanceclient==1.1.0",
                       "python-heatclient==0.8.0",
diff --git a/ci/verify.sh b/ci/verify.sh
index d309319..89b1406 100755
--- a/ci/verify.sh
+++ b/ci/verify.sh
@@ -21,7 +21,7 @@ source $WORKSPACE/storperf_venv/bin/activate
 pip install setuptools
 pip install autoflake==0.6.6
 pip install autopep8==1.2.2
-pip install coverage==3.4
+pip install coverage==4.1
 pip install flask==0.10
 pip install flask-restful==0.3.5
 pip install flask-restful-swagger==0.19
@@ -29,6 +29,7 @@ pip install flask-swagger==0.2.12
 pip install funcsigs==0.4
 pip install flake8==2.5.4
 pip install html2text==2016.1.8
+pip install matplotlib==1.3.1
 pip install mock==1.3.0
 pip install nose==1.3.7
 pip install python-cinderclient==1.6.0
@@ -45,6 +46,8 @@ python ci/setup.py develop
 
 flake8 storperf
 
+flake8rc=$?
+
 nosetests --with-xunit \
          --with-coverage \
          --cover-package=storperf\
@@ -54,4 +57,10 @@ rc=$?
 
 deactivate
 
-exit $rc
+if [ $flake8rc -ne 0 ]
+then
+    echo "Formatting did not meet guidelines"
+    exit $flake8rc
+fi
+
+exit $rc
\ No newline at end of file