From: Sébastien Han <seb@redhat.com>
Date: Tue, 3 Jul 2018 12:49:15 +0000 (+0200)
Subject: ceph-ansible-pr-syntax-check: force return 0
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F1070%2Fhead;p=ceph-build.git

ceph-ansible-pr-syntax-check: force return 0

If the test succeeds we return 0.

Signed-off-by: Sébastien Han <seb@redhat.com>
---

diff --git a/ceph-ansible-pr-syntax-check/build/build b/ceph-ansible-pr-syntax-check/build/build
index 3be3ec97..5b19523b 100644
--- a/ceph-ansible-pr-syntax-check/build/build
+++ b/ceph-ansible-pr-syntax-check/build/build
@@ -52,6 +52,9 @@ function test_sign_off {
   # shellcheck disable=SC2154
   # ghprbTargetBranch variable comes from jenkins's injectedEnvVars
   test "$(git log --oneline --no-merges origin/"${ghprbTargetBranch}"..HEAD | wc -l)" -ne "$(git log --no-merges origin/"${ghprbTargetBranch}"..HEAD | grep -c Signed-off-by)" && echo "One or more commits is/are missing a Signed-off-by. Add it with 'git commit -s'." && return 1
+
+  # if we arrive here the test command successed and we can return 0
+  echo "Sign-off ok!" && return 0
 }