[ARVADOS-DEV] updated: 35532d9eec8564a63e91af98a08117203e859d97

[git at public.curoverse.com]

Summary of changes:
 jenkins/run-build-packages.sh |  3 +-
 jenkins/run-tests.sh          | 82 ++++++++++++++++++++++++-------------------
 2 files changed, 48 insertions(+), 37 deletions(-)

       via  35532d9eec8564a63e91af98a08117203e859d97 (commit)
       via  1d1cdff657785ca3c9d930d1cf3ef462d98a1b24 (commit)
      from  45ab2f46c34ab5336768d095c0232f24efb6e04d (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 35532d9eec8564a63e91af98a08117203e859d97
Author: Tom Clegg <tom at curoverse.com>
Date:   Thu Aug 14 16:30:56 2014 -0400

    3551: Test and install keepstore before Python SDK.

diff --git a/jenkins/run-tests.sh b/jenkins/run-tests.sh
index bee8002..f52b585 100755
--- a/jenkins/run-tests.sh
+++ b/jenkins/run-tests.sh
@@ -17,6 +17,10 @@ export GOPATH=$(mktemp -d)
 mkdir -p "$GOPATH/src/git.curoverse.com"
 ln -sfn "$WORKSPACE" "$GOPATH/src/git.curoverse.com/arvados.git"
 
+VENVDIR=$(mktemp -d)
+virtualenv --setuptools "$VENVDIR"
+PATH="$VENVDIR/bin:$PATH"
+
 # DOCS
 title "Starting DOC build"
 cd "$WORKSPACE"
@@ -35,6 +39,17 @@ checkexit() {
     fi
 }
 
+gotest() {
+  title "Starting $1 tests"
+  cd "$WORKSPACE"
+
+  go get -t "git.curoverse.com/arvados.git/$1" \
+  && go test "git.curoverse.com/arvados.git/$1"
+
+  checkexit "$1 tests"
+  title "$1 tests complete"
+}
+
 checkexit "Doc build"
 title "DOC build complete"
 
@@ -98,53 +113,29 @@ bundle exec rake test
 checkexit "API server tests"
 title "API server tests complete"
 
-# Install and test Go bits. keepstore must come before keepproxy and keepclient.
-for dir in services/keepstore services/keepproxy sdk/go/arvadosclient sdk/go/keepclient sdk/go/streamer
-do
-  title "Starting $dir tests"
-  cd "$WORKSPACE"
-
-  go get -t "git.curoverse.com/arvados.git/$dir" \
-  && go test "git.curoverse.com/arvados.git/$dir"
-
-  checkexit "$dir tests"
-  title "$dir tests complete"
-done
-
-
-# WORKBENCH
-title "Starting workbench tests"
-cd "$WORKSPACE"
-cd apps/workbench
-bundle install --deployment
-
-echo $PATH
-
-
-bundle exec rake test
+# Keepstore. The keepstore binary is used by many other test suites,
+# so we test and install it early.
 
-checkexit "Workbench tests"
-title "Workbench tests complete"
+gotest services/keepstore
 
 # Python SDK
-title "Starting Python SDK tests"
+
 cd "$WORKSPACE"
 cd sdk/cli
 bundle install --deployment
 
 # Set up Python SDK and dependencies
 
+title "Starting Python SDK tests"
 cd "$WORKSPACE"
 cd sdk/python
 
-VENVDIR=$(mktemp -d)
-virtualenv --setuptools "$VENVDIR"
-"$VENVDIR/bin/python" setup.py test
+python setup.py test
 
 checkexit "Python SDK tests"
 
-"$VENVDIR/bin/python" setup.py egg_info -b ".$(git log --format=format:%ct.%h -n1 .)" sdist rotate --keep=1 --match .tar.gz
-"$VENVDIR/bin/pip" install dist/arvados-python-client-0.1.*.tar.gz
+python setup.py egg_info -b ".$(git log --format=format:%ct.%h -n1 .)" sdist rotate --keep=1 --match .tar.gz
+pip install dist/arvados-python-client-0.1.*.tar.gz
 
 checkexit "Python SDK install"
 
@@ -152,19 +143,38 @@ cd "$WORKSPACE"
 cd services/fuse
 
 # We reuse $VENVDIR from the Python SDK tests above
-"$VENVDIR/bin/python" setup.py test
+python setup.py test
 
 checkexit "FUSE tests"
 
-"$VENVDIR/bin/python" setup.py egg_info -b ".$(git log --format=format:%ct.%h -n1 .)" sdist rotate --keep=1 --match .tar.gz
-"$VENVDIR/bin/pip" install dist/arvados_fuse-0.1.*.tar.gz
+python setup.py egg_info -b ".$(git log --format=format:%ct.%h -n1 .)" sdist rotate --keep=1 --match .tar.gz
+pip install dist/arvados_fuse-0.1.*.tar.gz
 
 checkexit "FUSE install"
 
 title "Python SDK tests complete"
 
-# Clean up $VENVDIR
+for dir in services/keepproxy sdk/go/arvadosclient sdk/go/keepclient sdk/go/streamer
+do
+  gotest "$dir"
+end
+
+# WORKBENCH
+title "Starting workbench tests"
+cd "$WORKSPACE"
+cd apps/workbench
+bundle install --deployment
+
+echo $PATH
+
+bundle exec rake test
+
+checkexit "Workbench tests"
+title "Workbench tests complete"
+
+# Clean up temporary virtualenv and GOPATH
 rm -rf "$VENVDIR"
+rm -rf "$GOPATH"
 
 # The CLI SDK tests require a working API server, so let's skip those for now.
 exit $EXITCODE

commit 1d1cdff657785ca3c9d930d1cf3ef462d98a1b24
Author: Tom Clegg <tom at curoverse.com>
Date:   Thu Aug 14 16:30:13 2014 -0400

    3551: Clean up temp dir.

diff --git a/jenkins/run-build-packages.sh b/jenkins/run-build-packages.sh
index 65d0a9f..fb522f8 100755
--- a/jenkins/run-build-packages.sh
+++ b/jenkins/run-build-packages.sh
@@ -182,7 +182,6 @@ build_and_scp_deb $WORKSPACE/sdk/python python-arvados-python-client 'Curoverse,
 cd $WORKSPACE/services/fuse
 sed -i'' -e "s:version='0.1':version='0.1.$GIT_HASH':" setup.py
 
-
 cd $WORKSPACE/debs
 
 # Please seem comment about --no-python-fix-name above; we stay consistent and do
@@ -204,3 +203,5 @@ else
   echo "No new packages generated. No PRM run necessary."
 fi
 
+# clean up temporary GOPATH
+rm -rf "$GOPATH"

-----------------------------------------------------------------------


hooks/post-receive
--