You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "ASF subversion and git services (Jira)" <ji...@apache.org> on 2021/04/05 16:03:00 UTC

[jira] [Commented] (GEODE-8899) Update dockerized test plugin

    [ https://issues.apache.org/jira/browse/GEODE-8899?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17314945#comment-17314945 ] 

ASF subversion and git services commented on GEODE-8899:
--------------------------------------------------------

Commit d691567841363c28ba29cb5eb54fdbb77f4d4138 in geode's branch refs/heads/develop from Dale Emery
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=d691567 ]

GEODE-8899: Prep CI scripts for upcoming DUnit command line changes (#6254)

Change CI scripts to support running parallel DUnit tests with either
Geode's current Gradle 5.5 build or the upcoming Gradle 6.8.3 build.
This allows testing the upgrade to Gradle 6.8.3 in the PR precheckin
pipeline.

Where the current build uses a single command line option
(-PdunitParallelForks) to specify the maximum number of parallel
dockerized test workers, the upcoming build uses two (--max-workers and
-PtestMaxParallelForks).

This commit changes the CI build scripts to specify all three options on
the Gradle command line for parallel DUnit tests. With this commit:

- If the scripts run a Gradle 5.5 build, the old option will configure
  the tests, and the new options will have no effect.

- If the scripts run a Gradle 6.8.3 build, the new options will
  configure the tests, and the old option will have no effect.

Authored-by: Dale Emery <de...@vmware.com>

> Update dockerized test plugin
> -----------------------------
>
>                 Key: GEODE-8899
>                 URL: https://issues.apache.org/jira/browse/GEODE-8899
>             Project: Geode
>          Issue Type: Test
>          Components: tests
>            Reporter: Dale Emery
>            Assignee: Dale Emery
>            Priority: Major
>              Labels: GeodeOperationAPI, pull-request-available
>
> *Problem:* The dockerized test plugin includes modified copies of internal Gradle code. This code depends on internal Gradle classes and method signatures as defined in Gradle 5.8. These dependencies prevent us from updating Gradle.
> *Solution:* Update the dockerized test plugin code to be compatible with the latest Gradle, and to take advantage of recent Gradle features (such as the "worker lease" mechanism) that make some plugin details unnecessary.
> Where possible, change the plugin code to use public Gradle APIs.
> Where the plugin must depend on internal Gradle implementation details, comment the code to indicate:
>  * Which internal Gradle implementation details the code depends on
>  * The latest version of Gradle known to satisfy these dependencies
> Where the plugin must use modified copies of internal Gradle code, comment the code to indicate:
>  * Which Gradle class was copied
>  * Which version of Gradle the class was copied from
>  * What modifications were made in the copy, and why
> These comments will help identify implementation details in the plugin that risk incompatibility with future versions of Gradle.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)