----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60142/#review178086 -----------------------------------------------------------
Ship it! Excellent work Jens! Only issue I see is to add Apache license to docker.gradle. - Mark Bretl On June 15, 2017, 5:59 p.m., Jens Deppe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60142/ > ----------------------------------------------------------- > > (Updated June 15, 2017, 5:59 p.m.) > > > Review request for geode, Anthony Baker, Mark Bretl, and Dan Smith. > > > Repository: geode > > > Description > ------- > > Herewith the ability to leverage Gradle's parallel test execution > capability to run dunits in parallel. This is combined with launching > tests in Docker containers to provide process, network and filesystem > isolation. Depending on the size of your system, this can speed up > running the distributedTest task 2-5 times. > > The capability is enabled by launching gradle with '-PparallelDunit' > > Tunables, enabled as gradle parametrs (-P option) are: > > - dunitDockerImage: The docker image which will be used to launch > tests. The image must have the JAVA_HOME environment variable set. The > image must be pulled locally before starting the tests. > - dunitParallelForks: The number of parallel docker containers to be > launched. > - dunitDockerUser: The docker user which will run the tests. Because of > the way that the containers map the build directory into them, the > test artifacts, will be written with this user id. By default this is > 'root'. > > Remove debug println > > NOTE: There are problems running this on MacOS which will require a bit more > work. > > > Diffs > ----- > > build.gradle ec6b920c825491030120e99d288067d476d1fdcb > gradle.properties ca79a3816bffa56e412d82dd570f1e0e445592be > gradle/docker.gradle PRE-CREATION > > > Diff: https://reviews.apache.org/r/60142/diff/1/ > > > Testing > ------- > > Manual test on linux > > > Thanks, > > Jens Deppe > >
