You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by si...@apache.org on 2018/06/05 06:53:24 UTC

[bookkeeper] branch master updated: Throttle integration tests to run 1 build per host

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

sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/master by this push:
     new ecf5de6  Throttle integration tests to run 1 build per host
ecf5de6 is described below

commit ecf5de6af7f90a3bf5a8a65177bff6078995112c
Author: Sijie Guo <si...@apache.org>
AuthorDate: Mon Jun 4 23:53:16 2018 -0700

    Throttle integration tests to run 1 build per host
    
    Descriptions of the changes in this PR:
    
    If running multiple integration tests build on same host, they will be conflicting on installing docker images. So throttle integration tests to run max 1 build per host.
    
    See discussions at : http://mail-archives.apache.org/mod_mbox/pulsar-dev/201806.mbox/%3CCAJdLeK0nj9n-TKC4t43Nyc8bO9UUW17wM%2By_fF68EgFL6Fpo8w%40mail.gmail.com%3E
    
    Author: Sijie Guo <si...@apache.org>
    
    Reviewers: Enrico Olivelli <eo...@gmail.com>
    
    This closes #1480 from sijie/disable_concurrent_builds
---
 .../jenkins/job_bookkeeper_precommit_integrationtests.groovy        | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/.test-infra/jenkins/job_bookkeeper_precommit_integrationtests.groovy b/.test-infra/jenkins/job_bookkeeper_precommit_integrationtests.groovy
index 9cd7635..b818d1f 100644
--- a/.test-infra/jenkins/job_bookkeeper_precommit_integrationtests.groovy
+++ b/.test-infra/jenkins/job_bookkeeper_precommit_integrationtests.groovy
@@ -31,8 +31,10 @@ freeStyleJob('bookkeeper_precommit_integrationtests') {
         'ubuntu',
         '${ghprbTargetBranch}')
 
-    // Execute concurrent builds if necessary.
-    concurrentBuild()
+    throttleConcurrentBuilds {
+        // limit builds to 1 per node to avoid conflicts on building docker images
+        maxPerNode(1)
+    }
 
     // Sets that this is a PreCommit job.
     common_job_properties.setPreCommit(delegate, 'Integration Tests')

-- 
To stop receiving notification emails like this one, please contact
sijie@apache.org.