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

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


The following commit(s) were added to refs/heads/master by this push:
     new fb9f5f4  Fix scheduling for jobs
fb9f5f4 is described below

commit fb9f5f44d53c01de18484f90fe4f22e80f65062c
Author: Mikhail Gryzykhin <mig...@google.com>
AuthorDate: Tue Jul 31 14:39:33 2018 -0700

    Fix scheduling for jobs
---
 .test-infra/jenkins/CommonJobProperties.groovy  | 4 ++--
 .test-infra/jenkins/PostcommitJobBuilder.groovy | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/.test-infra/jenkins/CommonJobProperties.groovy 
b/.test-infra/jenkins/CommonJobProperties.groovy
index a804304..aa858ff 100644
--- a/.test-infra/jenkins/CommonJobProperties.groovy
+++ b/.test-infra/jenkins/CommonJobProperties.groovy
@@ -224,9 +224,9 @@ class CommonJobProperties {
 
   // Sets common config for jobs which run on a schedule; optionally on push
   static void setAutoJob(context,
-                         triggerOnCommit = false,
                          String buildSchedule = '0 */6 * * *',
-                         notifyAddress = 'commits@beam.apache.org') {
+                         notifyAddress = 'commits@beam.apache.org',
+                         triggerOnCommit = false) {
 
     // Set build triggers
     context.triggers {
diff --git a/.test-infra/jenkins/PostcommitJobBuilder.groovy 
b/.test-infra/jenkins/PostcommitJobBuilder.groovy
index 3235582..3b53044 100644
--- a/.test-infra/jenkins/PostcommitJobBuilder.groovy
+++ b/.test-infra/jenkins/PostcommitJobBuilder.groovy
@@ -47,7 +47,7 @@ class PostcommitJobBuilder {
 
   void defineAutoPostCommitJob(name) {
     def autoBuilds = scope.job(name) {
-      commonJobProperties.setAutoJob delegate, true
+      commonJobProperties.setAutoJob delegate, '0 */6 * * *', 
'commits@beam.apache.org', true
     }
 
     autoBuilds.with(jobDefinition)

Reply via email to