Expand CI integration

* make sure jenkins.sh fails on any errors similar to other jenkins jobs
* always explicitly use python2 instead of generic python
* add comments

Change-Id: I0f4639537d227c513859d4552533ce1e41df9deb
---
M contrib/jenkins.sh
1 file changed, 8 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/python/osmo-python-tests 
refs/changes/77/5277/2

diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 802ba2d..eafc2f4 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -1,4 +1,11 @@
 #!/bin/sh
-python2 ./setup.py install || python ./setup.py install
+
+set -ex
+
+# FIXME: remove once python 2 support is deprecated
+python2 ./setup.py install
+
 rm -rf ./build
 python3 ./setup.py install
+
+# TODO: add more tests

-- 
To view, visit https://gerrit.osmocom.org/5277
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I0f4639537d227c513859d4552533ce1e41df9deb
Gerrit-PatchSet: 2
Gerrit-Project: python/osmo-python-tests
Gerrit-Branch: master
Gerrit-Owner: Max <msur...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Neels Hofmeyr <nhofm...@sysmocom.de>
Gerrit-Reviewer: Pau Espin Pedrol <pes...@sysmocom.de>

Reply via email to