This is an automated email from the ASF dual-hosted git repository.

driazati pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git


The following commit(s) were added to refs/heads/main by this push:
     new 55dcd5f195 [ci] Don't skip index-triggered builds (#11915)
55dcd5f195 is described below

commit 55dcd5f195b15401b79f280d4146ddbf75806b7c
Author: driazati <[email protected]>
AuthorDate: Fri Jul 1 14:45:02 2022 -0700

    [ci] Don't skip index-triggered builds (#11915)
    
    This code was there to stop Jenkins restarts from doing a repository scan 
and scheduling a ton of builds. However, I haven't noticed this happening 
during restarts lately, and repository scans are useful to patch up PRs that 
didn't get CI run properly (i.e. while Jenkins was down or something).
    
    For example in #11914 since this code is there all the messed up PRs needed 
their CI to be manually re-triggered even though they were detected during the 
scan.
---
 Jenkinsfile               | 10 +---------
 ci/jenkins/Jenkinsfile.j2 |  8 --------
 2 files changed, 1 insertion(+), 17 deletions(-)

diff --git a/Jenkinsfile b/Jenkinsfile
index 513f11eaaf..8b59fe2192 100755
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -45,7 +45,7 @@
 // 'python3 jenkins/generate.py'
 // Note: This timestamp is here to ensure that updates to the Jenkinsfile are
 // always rebased on main before merging:
-// Generated at 2022-06-27T17:30:37.779354
+// Generated at 2022-07-01T12:43:52.727636
 
 import org.jenkinsci.plugins.pipeline.modeldefinition.Utils
 // NOTE: these lines are scanned by docker/dev_common.sh. Please update the 
regex as needed. -->
@@ -86,14 +86,6 @@ docker_build = 'docker/build.sh'
 max_time = 180
 rebuild_docker_images = false
 
-// skips builds from branch indexing; sourced from 
https://www.jvt.me/posts/2020/02/23/jenkins-multibranch-skip-branch-index/
-// execute this before anything else, including requesting any time on an agent
-if (currentBuild.getBuildCauses().toString().contains('BranchIndexingCause')) {
-  print "INFO: Build skipped due to trigger being Branch Indexing"
-  currentBuild.result = 'ABORTED' // optional, gives a better hint to the user 
that it's been skipped, rather than the default which shows it's successful
-  return
-}
-
 // Filenames for stashing between build and test steps
 s3_prefix = 
"tvm-jenkins-artifacts-prod/tvm/${env.BRANCH_NAME}/${env.BUILD_NUMBER}"
 
diff --git a/ci/jenkins/Jenkinsfile.j2 b/ci/jenkins/Jenkinsfile.j2
index 22cd5b6bfc..2c38bf32c6 100644
--- a/ci/jenkins/Jenkinsfile.j2
+++ b/ci/jenkins/Jenkinsfile.j2
@@ -83,14 +83,6 @@ docker_build = 'docker/build.sh'
 max_time = 180
 rebuild_docker_images = false
 
-// skips builds from branch indexing; sourced from 
https://www.jvt.me/posts/2020/02/23/jenkins-multibranch-skip-branch-index/
-// execute this before anything else, including requesting any time on an agent
-if (currentBuild.getBuildCauses().toString().contains('BranchIndexingCause')) {
-  print "INFO: Build skipped due to trigger being Branch Indexing"
-  currentBuild.result = 'ABORTED' // optional, gives a better hint to the user 
that it's been skipped, rather than the default which shows it's successful
-  return
-}
-
 // Filenames for stashing between build and test steps
 {% set tvm_runtime = ['build/libtvm_runtime.so', 'build/config.cmake'] %}
 {% set tvm_lib = ['build/libtvm.so'] + tvm_runtime %}

Reply via email to