You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by go...@apache.org on 2019/05/14 17:18:58 UTC
[beam] branch master updated: Fix trigger paths filters to execute
precommit tests only if needed
This is an automated email from the ASF dual-hosted git repository.
goenka 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 7e3559b Fix trigger paths filters to execute precommit tests only if needed
new a648016 Merge pull request #8483 from iemejia/fix-precommits-triggers
7e3559b is described below
commit 7e3559bbebc0c62eaec4dd21ce065b3a4f1a63fc
Author: Ismaël Mejía <ie...@gmail.com>
AuthorDate: Fri May 3 15:55:34 2019 +0200
Fix trigger paths filters to execute precommit tests only if needed
---
.test-infra/jenkins/job_PreCommit_Go.groovy | 1 -
.../jenkins/job_PreCommit_Portable_Python.groovy | 7 +++++-
.test-infra/jenkins/job_PreCommit_Python.groovy | 1 -
...b_PreCommit_Python_ValidatesRunner_Flink.groovy | 26 +++++++++++++---------
4 files changed, 21 insertions(+), 14 deletions(-)
diff --git a/.test-infra/jenkins/job_PreCommit_Go.groovy b/.test-infra/jenkins/job_PreCommit_Go.groovy
index e2b82c3..ac121b5 100644
--- a/.test-infra/jenkins/job_PreCommit_Go.groovy
+++ b/.test-infra/jenkins/job_PreCommit_Go.groovy
@@ -25,7 +25,6 @@ PrecommitJobBuilder builder = new PrecommitJobBuilder(
triggerPathPatterns: [
'^model/.*$',
'^sdks/go/.*$',
- '^runners/.*$',
'^release/.*$',
]
)
diff --git a/.test-infra/jenkins/job_PreCommit_Portable_Python.groovy b/.test-infra/jenkins/job_PreCommit_Portable_Python.groovy
index 1f26898..0dd3b34 100644
--- a/.test-infra/jenkins/job_PreCommit_Portable_Python.groovy
+++ b/.test-infra/jenkins/job_PreCommit_Portable_Python.groovy
@@ -24,7 +24,12 @@ PrecommitJobBuilder builder = new PrecommitJobBuilder(
gradleTask: ':portablePythonPreCommit',
triggerPathPatterns: [
'^model/.*$',
- '^runners/.*$',
+ '^runners/core-construction-java/.*$',
+ '^runners/core-java/.*$',
+ '^runners/extensions-java/.*$'
+ '^runners/flink/.*$',
+ '^runners/java-fn-execution/.*$',
+ '^runners/reference/.*$'
'^sdks/python/.*$',
'^release/.*$',
]
diff --git a/.test-infra/jenkins/job_PreCommit_Python.groovy b/.test-infra/jenkins/job_PreCommit_Python.groovy
index 35b34f2..00fc6e1 100644
--- a/.test-infra/jenkins/job_PreCommit_Python.groovy
+++ b/.test-infra/jenkins/job_PreCommit_Python.groovy
@@ -24,7 +24,6 @@ PrecommitJobBuilder builder = new PrecommitJobBuilder(
gradleTask: ':pythonPreCommit',
triggerPathPatterns: [
'^model/.*$',
- '^runners/.*$',
'^sdks/python/.*$',
'^release/.*$',
]
diff --git a/.test-infra/jenkins/job_PreCommit_Python_ValidatesRunner_Flink.groovy b/.test-infra/jenkins/job_PreCommit_Python_ValidatesRunner_Flink.groovy
index 086578c..69ae1d2 100644
--- a/.test-infra/jenkins/job_PreCommit_Python_ValidatesRunner_Flink.groovy
+++ b/.test-infra/jenkins/job_PreCommit_Python_ValidatesRunner_Flink.groovy
@@ -1,4 +1,3 @@
-
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
@@ -21,16 +20,21 @@ import PrecommitJobBuilder
// This job runs the suite of ValidatesRunner tests against the Flink runner.
PrecommitJobBuilder builder = new PrecommitJobBuilder(
- scope: this,
- nameBase: 'Python_PVR_Flink',
- gradleTask: ':sdks:python:flinkValidatesRunner',
- triggerPathPatterns: [
- '^model/.*$',
- '^runners/.*$',
- '^sdks/python/.*$',
- '^release/.*$',
- ]
+ scope: this,
+ nameBase: 'Python_PVR_Flink',
+ gradleTask: ':beam-sdks-python:flinkValidatesRunner',
+ triggerPathPatterns: [
+ '^model/.*$',
+ '^runners/core-construction-java/.*$',
+ '^runners/core-java/.*$',
+ '^runners/extensions-java/.*$'
+ '^runners/flink/.*$',
+ '^runners/java-fn-execution/.*$',
+ '^runners/reference/.*$',
+ '^sdks/python/.*$',
+ '^release/.*$',
+ ]
)
builder.build {
- previousNames('beam_PostCommit_Python_VR_Flink')
+ previousNames('beam_PostCommit_Python_VR_Flink')
}