This is an automated email from the ASF dual-hosted git repository.
upthewaterspout pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/develop by this push:
new e0b1725 GEODE-5893: Keep docker configuration in RepeatTest
e0b1725 is described below
commit e0b172591bb1cc2af604fa5eefe4d961c6562ff4
Author: Dan Smith <[email protected]>
AuthorDate: Wed Oct 17 15:23:24 2018 -0700
GEODE-5893: Keep docker configuration in RepeatTest
The RepeatTest task was not retaining the workerProcessFactory installed
by the docker plugin, because the new version of the docker plugin used
a different implementation of TestExcecutor.
---
buildSrc/src/main/groovy/org/apache/geode/gradle/RepeatTest.groovy | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/buildSrc/src/main/groovy/org/apache/geode/gradle/RepeatTest.groovy
b/buildSrc/src/main/groovy/org/apache/geode/gradle/RepeatTest.groovy
index a510761..7123101 100644
--- a/buildSrc/src/main/groovy/org/apache/geode/gradle/RepeatTest.groovy
+++ b/buildSrc/src/main/groovy/org/apache/geode/gradle/RepeatTest.groovy
@@ -48,14 +48,10 @@ class RepeatTest extends Test {
protected TestExecuter<JvmTestExecutionSpec> createTestExecuter() {
def oldExecutor = super.createTestExecuter()
- def workerProcessFactory = getProcessBuilderFactory()
-
//Use the previously set worker process factory. If the test is
//being run using the parallel docker plugin, this will be a docker
//process factory
- if(oldExecutor instanceof DefaultTestExecuter) {
- workerProcessFactory = oldExecutor.workerFactory
- }
+ def workerProcessFactory = oldExecutor.workerFactory
return new OverriddenTestExecutor(workerProcessFactory, getActorFactory(),
getModuleRegistry(),
