From: David Galloway <dgallowa@redhat.com>
Date: Wed, 30 Sep 2020 16:24:36 +0000 (-0400)
Subject: ceph-pr-commits: Don't require signed commits for doc/ dir
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F1671%2Fhead;p=ceph-build.git

ceph-pr-commits: Don't require signed commits for doc/ dir

Signed-off-by: David Galloway <dgallowa@redhat.com>
---

diff --git a/ceph-pr-commits/build/build b/ceph-pr-commits/build/build
index 758d49fe1..a22a3c5f2 100644
--- a/ceph-pr-commits/build/build
+++ b/ceph-pr-commits/build/build
@@ -1,9 +1,25 @@
 #!/bin/bash
 
+# Don't require signed commits if only docs changed.
+# Modified from ceph-docs/build/build.
+# I tried using the excluded-regions parameter for the ghprb plugin but since
+# this job/check is required, it hung with 'Expected — Waiting for status to be reported'
+cd "$WORKSPACE/ceph"
+if git rev-parse --verify ${GIT_COMMIT}^2; then
+     files="$(git diff --name-only ${GIT_COMMIT}^1...${GIT_COMMIT}^2)"
+     echo -e "changed files:\n$files"
+     if [ $(echo "$files" | grep -v '^doc/' | wc -l) -gt 0 ]; then
+         echo "Not a docs only change.  Will proceed with signed commit check."
+         cd -
+     else
+         echo "Only the doc/ dir changed.  No need to check for signed commits."
+         exit 0
+     fi
+fi
+
 # the following two methods exist in scripts/build_utils.sh
 pkgs=( "pytest" )
 install_python_packages "pkgs[@]"
 
-
 cd "$WORKSPACE"
 timeout 600 $VENV/py.test -vs --junitxml="$WORKSPACE/report.xml" "$WORKSPACE/ceph-build/ceph-pr-commits/build/test_commits.py"