From: Dan Mick <dmick@redhat.com>
Date: Thu, 23 May 2024 23:27:06 +0000 (-0700)
Subject: scripts/build_utils.sh: set DOCS_ONLY for PRs with only admin/ files, too
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F2244%2Fhead;p=ceph-build.git

scripts/build_utils.sh: set DOCS_ONLY for PRs with only admin/ files, too

DOCS_ONLY is set if the filenames all start with docs/, but it's
also true that if the filenames start with admin/, it's strictly
docs related (discovered for https://github.com/ceph/ceph/pull/57649)

Signed-off-by: Dan Mick <dmick@redhat.com>
---

diff --git a/scripts/build_utils.sh b/scripts/build_utils.sh
index 090159f3..5bbc4888 100755
--- a/scripts/build_utils.sh
+++ b/scripts/build_utils.sh
@@ -1705,7 +1705,7 @@ docs_pr_only() {
     files="$(git diff --name-only origin/${ghprbTargetBranch}...origin/pr/${ghprbPullId}/head)"
   fi
   echo -e "changed files:\n$files"
-  if [ $(echo "$files" | grep -v '^doc/' | wc -l) -gt 0 ]; then
+  if [ $(echo "$files" | egrep -v '^(doc/|admin/)' | wc -l) -gt 0 ]; then
       DOCS_ONLY=false
   else
       DOCS_ONLY=true