[GitHub] [geode] demery-pivotal opened a new pull request, #7572: GEODE-10225: Gfsh start commands add exports on JDK 11+

[GitBox <gi...@apache.org>]

demery-pivotal opened a new pull request, #7572:
URL: https://github.com/apache/geode/pull/7572

   PROBLEM
   
   When run on JDK 17, numerous acceptance tests fail because a launched
   locator or server requires access to the following packages:
   - java.base/sun.nio.ch
   - java.management/com.sun.jmx.remote.security
   
   SOLUTION
   
   - Add a `MemberJvmOptions` class that reports the JDK-dependent options
     required to launch a locator or server on the current process's JDK.
   - Change `StartLocatorCommand` and `StartServerCommand` to add those
     options to the command line when launching locators and servers.
   
   FUTURE
   
   Future PRs will likely put these opens/exports in an argument file, then
   configure these commands to use the argument file. We are deferring that
   until we identify additional needs for the argument file, so we can
   choose an appropriate location for the argument file.
   
   Co-authored-by: Dale Emery <de...@vmware.com>
   Co-authored-by: Kirk Lund <kl...@apache.org>
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] demery-pivotal merged pull request #7572: GEODE-10225: Gfsh start commands add exports on JDK 11+

[GitBox <gi...@apache.org>]

demery-pivotal merged PR #7572:
URL: https://github.com/apache/geode/pull/7572


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] demery-pivotal commented on pull request #7572: GEODE-10225: Gfsh start commands add exports on JDK 11+

[GitBox <gi...@apache.org>]

demery-pivotal commented on PR #7572:
URL: https://github.com/apache/geode/pull/7572#issuecomment-1093053570

   This PR fixes a common cause for many acceptance test failures on JDK 17.
   - The remaining acceptance test failures on JDK 17 have specific causes that will be fixed on other PRs.
   - The failures of other types of tests on JDK 17 are expected, and will be fixed on other PRs.
   - The InternalCacheForClientAccessDistributedTest that fails on JDK 8 is known to be unreliable: https://issues.apache.org/jira/browse/GEODE-10209
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org