Address some comments on PR/5048 (apache#19)

* Fix file permissions. * Delete dead test. Update postcommit project paths. Drop 'Gradle' from Python precommit PR trigger.
cosmoskitten · Apr 9, 2018 · c73672d · c73672d
1 parent ee69b4c
commit c73672d
Show file tree

Hide file tree

Showing 7 changed files with 5 additions and 65 deletions.
diff --git a/.test-infra/jenkins/job_beam_PostCommit_Java_JDKVersionsTest.groovy b/.test-infra/jenkins/job_beam_PostCommit_Java_JDKVersionsTest.groovy
diff --git a/.test-infra/jenkins/job_beam_PostCommit_Java_ValidatesRunner_Apex.groovy b/.test-infra/jenkins/job_beam_PostCommit_Java_ValidatesRunner_Apex.groovy
@@ -54,7 +54,7 @@ job('beam_PostCommit_Java_ValidatesRunner_Apex_Gradle') {
   steps {
     gradle {
       rootBuildScriptDir(common_job_properties.checkoutDir)
-      tasks(':runners:apex:validatesRunner')
+      tasks(':beam-runners-apex:validatesRunner')
       for (String gradle_switch : gradle_switches) {
         switches(gradle_switch)
       }

diff --git a/.test-infra/jenkins/job_beam_PostCommit_Java_ValidatesRunner_Dataflow.groovy b/.test-infra/jenkins/job_beam_PostCommit_Java_ValidatesRunner_Dataflow.groovy
@@ -55,7 +55,7 @@ job('beam_PostCommit_Java_ValidatesRunner_Dataflow_Gradle') {
   steps {
     gradle {
       rootBuildScriptDir(common_job_properties.checkoutDir)
-      tasks(':runners:google-cloud-dataflow-java:validatesRunner')
+      tasks(':beam-runners-google-cloud-dataflow-java:validatesRunner')
       for (String gradle_switch : gradle_switches) {
         switches(gradle_switch)
       }

diff --git a/.test-infra/jenkins/job_beam_PostCommit_Java_ValidatesRunner_Gearpump.groovy b/.test-infra/jenkins/job_beam_PostCommit_Java_ValidatesRunner_Gearpump.groovy
@@ -59,7 +59,7 @@ job('beam_PostCommit_Java_ValidatesRunner_Gearpump_Gradle') {
   steps {
     gradle {
       rootBuildScriptDir(common_job_properties.checkoutDir)
-      tasks(':runners:gearpump:validatesRunner')
+      tasks(':beam-runners-gearpump:validatesRunner')
       for (String gradle_switch : gradle_switches) {
         switches(gradle_switch)
       }

diff --git a/.test-infra/jenkins/job_beam_PostCommit_Java_ValidatesRunner_Spark.groovy b/.test-infra/jenkins/job_beam_PostCommit_Java_ValidatesRunner_Spark.groovy
@@ -54,7 +54,7 @@ job('beam_PostCommit_Java_ValidatesRunner_Spark_Gradle') {
   steps {
     gradle {
       rootBuildScriptDir(common_job_properties.checkoutDir)
-      tasks(':runners:spark:validatesRunner')
+      tasks(':beam-runners-spark:validatesRunner')
       for (String gradle_switch : gradle_switches) {
         switches(gradle_switch)
       }

diff --git a/.test-infra/jenkins/job_beam_PreCommit_Python_GradleBuild.groovy b/.test-infra/jenkins/job_beam_PreCommit_Python_GradleBuild.groovy
@@ -49,7 +49,7 @@ job('beam_PreCommit_Python_GradleBuild') {
 
   def gradle_command_line = './gradlew ' + gradle_switches.join(' ') + ' :pythonPreCommit'
   // Sets that this is a PreCommit job.
-  common_job_properties.setPreCommit(delegate, gradle_command_line, 'Run Python Gradle PreCommit')
+  common_job_properties.setPreCommit(delegate, gradle_command_line, 'Run Python PreCommit')
   steps {
     gradle {
       rootBuildScriptDir(common_job_properties.checkoutDir)

diff --git a/build_rules.gradle b/build_rules.gradle