Change in vdsm[master]: automation: Run make check over el7 in automation CI

ybronhei at redhat.com ybronhei at redhat.com
Thu Apr 14 08:04:07 UTC 2016


Yaniv Bronhaim has uploaded a new change for review.

Change subject: automation: Run make check over el7 in automation CI
......................................................................

automation: Run make check over el7 in automation CI

We tried to avoid running make check over el7 to save resources, but we
missed bugs and failed test that way. The timing implications of adding it back
are not so bad.

Change-Id: Iabbb09920de316a4e26447a0ca77ba9f30e517cd
Signed-off-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M automation/check-patch.sh
1 file changed, 5 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/22/56122/1

diff --git a/automation/check-patch.sh b/automation/check-patch.sh
index 4c43c5a..a907d02 100755
--- a/automation/check-patch.sh
+++ b/automation/check-patch.sh
@@ -6,11 +6,7 @@
 
 ./autogen.sh --system --enable-hooks
 
-# Run nosetests only over fedora mock to save Jenkins resources
-if grep -q 'Fedora' /etc/redhat-release; then
-   make check NOSE_WITH_COVERAGE=1 NOSE_COVER_PACKAGE="$PWD/vdsm,$PWD/lib"
-fi
-
+make check NOSE_WITH_COVERAGE=1 NOSE_COVER_PACKAGE="$PWD/vdsm,$PWD/lib"
 ./automation/build-artifacts.sh
 
 # enable complex globs
@@ -20,9 +16,7 @@
     yum -y install "$EXPORT_DIR/"!(*.src).rpm
 fi
 
-if grep -q 'Fedora' /etc/redhat-release; then
-   # Generate coverage report in HTML format
-   pushd tests
-   coverage html -d "$EXPORT_DIR/htmlcov"
-   popd
-fi
+# Generate coverage report in HTML format
+pushd tests
+coverage html -d "$EXPORT_DIR/htmlcov"
+popd


-- 
To view, visit https://gerrit.ovirt.org/56122
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iabbb09920de316a4e26447a0ca77ba9f30e517cd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei at redhat.com>


More information about the vdsm-patches mailing list