From: Guillaume Abrioux Date: Thu, 13 Jul 2017 12:02:22 +0000 (+0200) Subject: Add timeout for ceph-docker X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F786%2Fhead;p=ceph-build.git Add timeout for ceph-docker - Add the same timeout than for ceph-ansible PRs in ceph-docker PRs. - Change logged error message Signed-off-by: Guillaume Abrioux --- diff --git a/ceph-ansible-prs/build/build b/ceph-ansible-prs/build/build index 285c0167..bb8b1d1f 100644 --- a/ceph-ansible-prs/build/build +++ b/ceph-ansible-prs/build/build @@ -15,7 +15,7 @@ restart_libvirt_services # the $SCENARIO var is injected by the job template. It maps # to an actual, defined, tox environment -if ! timeout 5h $VENV/tox -rv -e=$RELEASE-$SCENARIO --workdir=$WORKDIR -- --provider=libvirt; then - echo "ERROR: Job didn't complete before the timeout exceeded (5h)." +if ! timeout 3h $VENV/tox -rv -e=$RELEASE-$SCENARIO --workdir=$WORKDIR -- --provider=libvirt; then + echo "ERROR: Job didn't complete successfully or got stuck for more than 3h." exit 1 fi diff --git a/ceph-docker-prs/build/build b/ceph-docker-prs/build/build index 8b9adb4b..6eabf8e4 100644 --- a/ceph-docker-prs/build/build +++ b/ceph-docker-prs/build/build @@ -22,4 +22,7 @@ restart_libvirt_services # adding groups on the fly doesn't guarantee their availability # so we must use `sg` to execute the tests as part of the docker group to avoid # 'Permission Denied` when tryin to talk over the socket -sg docker -c "$VENV/tox -rv -e=$SCENARIO --workdir=$WORKDIR" +if ! timeout 3h sg docker -c "$VENV/tox -rv -e=$SCENARIO --workdir=$WORKDIR"; then + echo "ERROR: Job didn't complete successfully or got stuck for more than 3h." + exit 1 +fi