Support for please build PR tests i.e. not running Relvals/Addons or other tests

pr-schedule-tests

@@ -67,7 +67,12 @@ if [ $(echo $CONFIG_LINE | tr ';' '\n' | grep SCRAM_ARCH= | wc -l) -eq 1 ] ; the
     fi
     if [ "${DOCKER_IMGAGE}" != "" ] ; then  DOCKER_IMG="${DOCKER_IMGAGE}" ; fi
     PR_COMMIT_STATUS="optional"
-    if ${REQUIRED_TEST} ; then PR_COMMIT_STATUS="required" ; fi
+    if [ "${BUILD_ONLY}" = "true" ] ; then
+      PR_COMMIT_STATUS="build_only"
+      REQUIRED_TEST=false
+    elif $REQUIRED_TEST ; then
+     PR_COMMIT_STATUS="required"
+    fi
     echo "REQUIRED_TEST=${REQUIRED_TEST}" >> $OUTPUT_FILE
     echo "DRY_RUN=${DRY_RUN}" >> $OUTPUT_FILE
     echo "ENABLE_BOT_TESTS=${ENABLE_BOT_TESTS}" >> $OUTPUT_FILE
@@ -118,6 +123,7 @@ if [ $(echo $CONFIG_LINE | tr ';' '\n' | grep SCRAM_ARCH= | wc -l) -eq 1 ] ; the
     echo "CONTEXT_PREFIX=${CONTEXT_PREFIX}" >> $OUTPUT_FILE
     echo "PROFILING_WORKFLOWS=${PROFILING_WORKFLOWS}" >> $OUTPUT_FILE
     echo "BUILD_VERBOSE=${BUILD_VERBOSE}" >> $OUTPUT_FILE
+    echo "BUILD_ONLY=${BUILD_ONLY}" >> $OUTPUT_FILE
     CMSSW_QUEUE="${RELEASE_QUEUE}"
     mark_commit_status_all_prs "${PR_COMMIT_STATUS}" 'pending' -d "Tests scheduled ${RELEASE_FORMAT}/${SCRAM_ARCH}" -u 'https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/' || true
     COMMIT_STATUS_CONTEXT="unknown/release"

pr_testing/test_multiple_prs.sh

@@ -119,6 +119,7 @@ PR_REPO=$(echo ${PULL_REQUEST} | sed 's|#.*||')
 PR_NUM=$(echo ${PULL_REQUEST} | md5sum | sed 's| .*||' | cut -c27-33)
 UPLOAD_UNIQ_ID=PR-${PR_NUM}/${BUILD_NUMBER}
 PR_RESULT_URL="https://cmssdt.cern.ch/SDT/${JENKINS_PREFIX}-artifacts/pull-request-integration/${UPLOAD_UNIQ_ID}"
+PR_COMMENT_TEXT_URL="https://cmssdt.cern.ch/SDT/cgi-bin/get_pr_results/${JENKINS_PREFIX}-artifacts/pull-request-integration/${UPLOAD_UNIQ_ID}/pr-result"
 NCPU=$(${COMMON}/get_cpu_number.sh)
 if [[  $NODE_NAME == *"cms-cmpwg-0"* ]]; then
    let NCPU=${NCPU}/2
@@ -185,6 +186,9 @@ for ex_type in $(echo ${ENABLE_BOT_TESTS} | tr "," " ") ; do
   fi
 done
 
+if [ "${BUILD_ONLY}" = "true" ] ; then
+  DO_COMPARISON=false
+fi
 # ----------
 # -- MAIN --
 # ----------
@@ -240,7 +244,12 @@ cp ${WORKSPACE}/prs_commits ${WORKSPACE}/prs_commits.txt
 
 mark_commit_status_all_prs '' 'pending' -u "${BUILD_URL}" -d 'Setting up build environment' --reset
 PR_COMMIT_STATUS="optional"
-if $REQUIRED_TEST ; then PR_COMMIT_STATUS="required" ; fi
+if [ "${BUILD_ONLY}" = "true" ] ; then
+  PR_COMMIT_STATUS="build_only"
+  export REQUIRED_TEST=false
+elif $REQUIRED_TEST ; then
+  PR_COMMIT_STATUS="required"
+fi
 mark_commit_status_all_prs "${PR_COMMIT_STATUS}" 'success' -d 'OK' -u "${BUILD_URL}"
 
 echo -n "**Summary**: ${PR_RESULT_URL}/summary.html" > ${RESULTS_DIR}/09-report.res
@@ -1323,7 +1332,7 @@ if [ -f $WORKSPACE/buildClang.log ] ; then
   fi
 fi
 
-mark_commit_status_all_prs '' 'pending' -u "${BUILD_URL}" -d "Running tests" || true
+[ "$BUILD_ONLY" = "true" ] && RUN_TESTS=false
 
 DO_PROFILING=false
 DO_GPU_TESTS=false
@@ -1420,7 +1429,6 @@ popd
 
 prepare_upload_results
 rm -rf $WORKSPACE/upload
-mark_commit_status_all_prs "${PR_COMMIT_STATUS}" 'success' -d 'OK' -u "${BUILD_URL}"
 
 echo "PR_BUILD_BASE=${WORKSPACE}" > $WORKSPACE/deploy-cmssw
 echo "CMS_WEEK=${CMS_WEEKLY_REPO}" >> $WORKSPACE/deploy-cmssw
@@ -1431,13 +1439,26 @@ echo "PULL_REQUEST=${PULL_REQUEST}" >> $WORKSPACE/deploy-cmssw
 echo "PULL_REQUESTS=${PULL_REQUESTS}" >> $WORKSPACE/deploy-cmssw
 echo "RELEASE_FORMAT=$CMSSW_IB" >> $WORKSPACE/deploy-cmssw
 
+if [ "${BUILD_ONLY}" = "true" ]; then
+  if ${ALL_OK} ; then
+    echo "+1" > comment.txt
+  else
+    echo "-1" > comment.txt
+  fi
+  echo "" >> comment.txt
+  curl -L ${PR_COMMENT_TEXT_URL} >> comment.txt
+  $WORKSPACE/cms-bot/comment-gh-pr.py --repository ${PR_REPO} --pullrequest ${PR_NUMBER} --report-file comment.txt
+  mark_commit_status_all_prs "${PR_COMMIT_STATUS}" 'success' -u "${BUILD_URL}" -d "Finished" -e
+fi
+
 if [ "X$BUILD_OK" != Xtrue -o "$RUN_TESTS" != "true" ]; then exit 0 ; fi
 
 touch $WORKSPACE/job.env
 for x in REPORT_OPTS BUILD_EXTERNAL DO_DUPLICATE_CHECKS DO_DAS_QUERY DO_TESTS CMSDIST_ONLY CMSSW_IB UPLOAD_UNIQ_ID PRODUCTION_RELEASE; do
   eval echo "$x=\\\"$(echo \$$x)\\\"" >> $WORKSPACE/job.env
 done
 
+
 echo "UPLOAD_UNIQ_ID=${UPLOAD_UNIQ_ID}" > $WORKSPACE/test-env.txt
 echo "CMSSW_CVMFS_PATH=/cvmfs/cms-ci.cern.ch/week${WEEK_NUM}/${PR_REPO}/${PR_NUMBER}/${BUILD_NUMBER}/${CMSSW_VERSION}" >> $WORKSPACE/test-env.txt
 echo "PULL_REQUEST=${PULL_REQUEST}" >> $WORKSPACE/test-env.txt

process_pr.py

@@ -124,7 +124,7 @@ def format(s, **kwds):
     cmsorgs="|".join(EXTERNAL_REPOS),
 )
 TEST_REGEXP = format(
-    r"^\s*((@|)cmsbuild\s*[,]*\s+|)(please\s*[,]*\s+|)test(\s+workflow(s|)\s+(%(workflow)s(\s*,\s*%(workflow)s|)*)|)(\s+with\s+(%(cms_pr)s(\s*,\s*%(cms_pr)s)*)|)(\s+for\s+%(release_queue)s|)(\s+using\s+full\s+cmssw|\s+using\s+(cms-|)addpkg\s+(%(pkg)s(,%(pkg)s)*)|)\s*$",
+    r"^\s*((@|)cmsbuild\s*[,]*\s+|)(please\s*[,]*\s+|)(test|build)(\s+workflow(s|)\s+(%(workflow)s(\s*,\s*%(workflow)s|)*)|)(\s+with\s+(%(cms_pr)s(\s*,\s*%(cms_pr)s)*)|)(\s+for\s+%(release_queue)s|)(\s+using\s+full\s+cmssw|\s+using\s+(cms-|)addpkg\s+(%(pkg)s(,%(pkg)s)*)|)\s*$",
     workflow=WF_PATTERN,
     cms_pr=CMS_PR_PATTERN,
     pkg=CMSSW_PACKAGE_PATTERN,
@@ -358,6 +358,8 @@ def extract_bot_cache(comment_msgs):
 
     data = ""
     for comment_msg in comment_msgs:
+        if not comment_msg.body:
+            continue
         seen_commits_match = REGEX_COMMITS_CACHE.search(comment_msg.body)
         if seen_commits_match:
             data += seen_commits_match[1]
@@ -565,9 +567,9 @@ def has_user_emoji(bot_cache, comment, repository, emoji, user):
         # github_utils.get_comment_emojis -> https://github.com/PyGithub/PyGithub/blob/v1.56/github/IssueComment.py#L135
         emojis = comment.get_reactions()
         for x in emojis:
-            if x["user"]["login"].encode("ascii", "ignore").decode() == user:
-                e = x["content"]
-                bot_cache["emoji"][comment_id] = x
+            if x.user.login.encode("ascii", "ignore").decode() == user:
+                e = x.content
+                bot_cache["emoji"][comment_id] = e
                 break
     return e and e == emoji
 
@@ -745,19 +747,20 @@ def check_test_cmd(first_line, repo, params):
         prs = []
         cmssw_que = ""
         logger.debug("check_test_cmd: %s", m.groups())
-        if m.group(6):
-            wfs = ",".join(set(m.group(6).replace(" ", "").split(",")))
-        if m.group(11):
-            prs = get_prs_list_from_string(m.group(11), repo)
-        if m.group(20):
-            cmssw_que = m.group(20)
-        if m.group(25):
-            if "addpkg" in m.group(25):
-                params["EXTRA_CMSSW_PACKAGES"] = m.group(27).strip()
+        build_only = m.group(4) == "build"
+        if m.group(7):
+            wfs = ",".join(set(m.group(7).replace(" ", "").split(",")))
+        if m.group(12):
+            prs = get_prs_list_from_string(m.group(12), repo)
+        if m.group(21):
+            cmssw_que = m.group(21)
+        if m.group(26):
+            if "addpkg" in m.group(26):
+                params["EXTRA_CMSSW_PACKAGES"] = m.group(28).strip()
             else:
                 params["BUILD_FULL_CMSSW"] = "true"
-        return (True, " ".join(prs), wfs, cmssw_que)
-    return (False, "", "", "")
+        return (True, " ".join(prs), wfs, cmssw_que, build_only)
+    return (False, "", "", "", False)
 
 
 def get_prs_list_from_string(pr_string="", repo_string=""):
@@ -1078,6 +1081,7 @@ def process_pr(
     ok_too_many_files = False
     warned_too_many_files = False
     is_draft_pr = False
+    build_comment = None
 
     if issue.pull_request:
         pr = repo.get_pull(prId)
@@ -1665,9 +1669,22 @@ def process_pr(
 
             # Check if the someone asked to trigger the tests
             if valid_commenter:
-                ok, v2, v3, v4 = check_test_cmd(first_line, repository, global_test_params)
+                ok, v2, v3, v4, v5 = check_test_cmd(first_line, repository, global_test_params)
                 if ok:
-                    test_comment = comment
+                    build_comment = None
+                    if v5:
+                        if has_user_emoji(bot_cache, comment, repository, "+1", cmsbuild_user):
+                            continue
+                        if test_comment and (
+                            not has_user_emoji(
+                                bot_cache, test_comment, repository, "+1", cmsbuild_user
+                            )
+                        ):
+                            continue
+                        build_comment = comment
+                    else:
+                        test_comment = comment
+                        signatures["tests"] = "pending"
                     abort_test = None
                     cmssw_prs = v2
                     extra_wfs = ",".join(sorted(v3.split(",")))
@@ -1687,7 +1704,6 @@ def process_pr(
                         extra_wfs,
                     )
                     logger.debug("Comment message: %s", first_line)
-                    signatures["tests"] = "pending"
                     continue
                 elif REGEX_TEST_ABORT.match(first_line) and (signatures["tests"] == "pending"):
                     abort_test = comment
@@ -2136,15 +2152,14 @@ def process_pr(
                 for status in commit_statuses:
                     if not status.context.startswith(cms_status_prefix + "/"):
                         continue
-                    cdata = status.context.split("/")
-                    if cdata[-1] not in ["optional", "required"]:
+                    scontext, suffix = status.context.rsplit("/", 1)
+                    if suffix not in ["optional", "required"]:
                         continue
-                    if (cdata[-1] not in lab_stats) or (cdata[-1] == "required"):
-                        lab_stats[cdata[-1]] = []
-                    lab_stats[cdata[-1]].append("pending")
+                    if (suffix not in lab_stats) or (suffix == "required"):
+                        lab_stats[suffix] = []
+                    lab_stats[suffix].append("pending")
                     if status.state == "pending":
                         continue
-                    scontext = "/".join(cdata[:-1])
                     all_states = {}
                     result_url = ""
                     for s in [
@@ -2179,18 +2194,18 @@ def process_pr(
                                 )
                         continue
                     if "success" in all_states:
-                        lab_stats[cdata[-1]][-1] = "success"
+                        lab_stats[suffix][-1] = "success"
                     if "error" in all_states:
                         if [c for c in all_states["error"] if ("/opt/" not in c)]:
-                            lab_stats[cdata[-1]][-1] = "error"
+                            lab_stats[suffix][-1] = "error"
                     logger.info(
-                        "Final Status: status.context=%s cdata[-1]=%s lab_stats[cdata[-1]][-1]=%s status.description=%s",
+                        "Final Status: status.context=%s suffix=%s lab_stats[suffix][-1]=%s status.description=%s",
                         status.context,
-                        cdata[-1],
-                        lab_stats[cdata[-1]][-1],
+                        suffix,
+                        lab_stats[suffix][-1],
                         status.description,
                     )
-                    if (lab_stats[cdata[-1]][-1] != "pending") and (
+                    if (lab_stats[suffix][-1] != "pending") and (
                         not status.description.startswith("Finished")
                     ):
                         if result_url:
@@ -2208,7 +2223,7 @@ def process_pr(
                                 raise Exception("System-error: unable to get PR result")
                             if o and (not dryRun):
                                 res = "+1"
-                                if lab_stats[cdata[-1]][-1] == "error":
+                                if lab_stats[suffix][-1] == "error":
                                     res = "-1"
                                 res = "%s\n\n%s" % (res, o)
                                 issue.create_comment(res)
@@ -2314,7 +2329,7 @@ def process_pr(
         else:
             labels.append("comparison-pending")
 
-    if ("PULL_REQUESTS" in global_test_params) or cmssw_prs:
+    if (("PULL_REQUESTS" in global_test_params) or cmssw_prs) and (not build_comment):
         need_external = True
     # Now updated the labels.
     xlabs = ["backport", "urgent", "backport-ok", "compilation-warnings"]
@@ -2470,16 +2485,19 @@ def process_pr(
     global_test_params["EXTRA_RELVALS_TESTS"] = " ".join(
         [t.upper().replace("-", "_") for t in EXTRA_RELVALS_TESTS]
     )
+    global_test_params["BUILD_ONLY"] = build_comment is not None
 
     logger.debug("All Parameters: %s", global_test_params)
     # For now, only trigger tests for cms-sw/cmssw and cms-sw/cmsdist
     if create_test_property:
-        global_test_params["CONTEXT_PREFIX"] = cms_status_prefix
-        if trigger_test:
+        global_test_params["CONTEXT_PREFIX"] = (
+            cms_status_prefix if not build_comment else cms_status_prefix + "/build"
+        )
+        if trigger_test or build_comment:
             create_properties_file_tests(
                 repository, prId, global_test_params, dryRun, abort=False, repo_config=repo_config
             )
-            set_comment_emoji_cache(dryRun, bot_cache, test_comment, repository)
+            set_comment_emoji_cache(dryRun, bot_cache, build_comment or test_comment, repository)
         elif abort_test and bot_status and (not bot_status.description.startswith("Aborted")):
             if not has_user_emoji(bot_cache, abort_test, repository, "+1", cmsbuild_user):
                 create_properties_file_tests(

tests/PRActionData/TestProcessPr.test_abort.json

@@ -110,7 +110,8 @@
                 "PULL_REQUESTS": "iarspider-cmssw/cmssw#17 cms-sw/cms-bot#2134",
                 "RELEASE_FORMAT": "CMSSW_14_1_CPP20_X",
                 "EXTRA_RELVALS_TESTS": "THREADING GPU HIGH_STATS NANO CUDA ROCM",
-                "CONTEXT_PREFIX": "cms/17"
+                "CONTEXT_PREFIX": "cms/17",
+                "BUILD_ONLY": false
             }
         }
     },

tests/PRActionData/TestProcessPr.test_build_only.json

@@ -0,0 +1,50 @@
+[
+    {
+        "type": "load-bot-cache",
+        "data": {
+            "commits": {
+                "06336c8884bcfbc516cc69fd99b0a161bb3a14d5": {
+                    "files": [
+                        "RecoLocalCalo/EcalRecProducers/plugins/alpaka/EcalUncalibRecHitPhase2WeightsAlgoPortable.dev.cc"
+                    ],
+                    "squashed": false,
+                    "time": 1747049040
+                }
+            },
+            "emoji": {},
+            "last_seen_sha": "06336c8884bcfbc516cc69fd99b0a161bb3a14d5",
+            "signatures": {}
+        }
+    },
+    {
+        "type": "add-label",
+        "data": []
+    },
+    {
+        "type": "remove-label",
+        "data": []
+    },
+    {
+        "type": "edit-comment",
+        "data": "cms-bot internal usage<!-- bot cache: {\"commits\":{\"06336c8884bcfbc516cc69fd99b0a161bb3a14d5\":{\"files\":[\"RecoLocalCalo/EcalRecProducers/plugins/alpaka/EcalUncalibRecHitPhase2WeightsAlgoPortable.dev.cc\"],\"squashed\":false,\"time\":1747049040}},\"emoji\":{\"2872583904\":\"+1\"},\"last_seen_sha\":\"06336c8884bcfbc516cc69fd99b0a161bb3a14d5\",\"signatures\":{}} -->"
+    },
+    {
+        "type": "save-bot-cache",
+        "data": {
+            "commits": {
+                "06336c8884bcfbc516cc69fd99b0a161bb3a14d5": {
+                    "files": [
+                        "RecoLocalCalo/EcalRecProducers/plugins/alpaka/EcalUncalibRecHitPhase2WeightsAlgoPortable.dev.cc"
+                    ],
+                    "squashed": false,
+                    "time": 1747049040
+                }
+            },
+            "emoji": {
+                "2872583904": "+1"
+            },
+            "last_seen_sha": "06336c8884bcfbc516cc69fd99b0a161bb3a14d5",
+            "signatures": {}
+        }
+    }
+]

tests/PRActionData/TestProcessPr.test_cmsdist_start_tests.json

@@ -58,7 +58,8 @@
             "data": {
                 "PULL_REQUESTS": "iarspider-cmssw/cmsdist#1",
                 "EXTRA_RELVALS_TESTS": "THREADING GPU HIGH_STATS NANO CUDA ROCM",
-                "CONTEXT_PREFIX": "cms/1"
+                "CONTEXT_PREFIX": "cms/1",
+                "BUILD_ONLY": false
             }
         }
     },

tests/PRActionData/TestProcessPr.test_draft_pr_ready.json

@@ -70,7 +70,8 @@
             "data": {
                 "PULL_REQUESTS": "iarspider-cmssw/cmssw#21",
                 "EXTRA_RELVALS_TESTS": "THREADING GPU HIGH_STATS NANO CUDA ROCM",
-                "CONTEXT_PREFIX": "cms/21"
+                "CONTEXT_PREFIX": "cms/21",
+                "BUILD_ONLY": false
             }
         }
     },

tests/PRActionData/TestProcessPr.test_draft_pr_start_test.json

@@ -70,7 +70,8 @@
             "data": {
                 "PULL_REQUESTS": "iarspider-cmssw/cmssw#21",
                 "EXTRA_RELVALS_TESTS": "THREADING GPU HIGH_STATS NANO CUDA ROCM",
-                "CONTEXT_PREFIX": "cms/21"
+                "CONTEXT_PREFIX": "cms/21",
+                "BUILD_ONLY": false
             }
         }
     },

tests/PRActionData/TestProcessPr.test_ignore_smth.json

@@ -67,7 +67,8 @@
                     null
                 ],
                 "EXTRA_RELVALS_TESTS": "THREADING GPU HIGH_STATS NANO CUDA ROCM",
-                "CONTEXT_PREFIX": "cms/25"
+                "CONTEXT_PREFIX": "cms/25",
+                "BUILD_ONLY": false
             }
         }
     },