Re: Flaky test caused by missing JDK dependency

2020-07-17 Thread Kirk Lund
Closing out this discussion thread about GEODE-6183: We believe that the machine performed an update of Java during the test. This caused the tools.jar to be unavailable only while this test was executing. There are many other tests that use the Attach API which passed in this overall CI job

Re: Flaky test caused by missing JDK dependency

2020-07-08 Thread Kirk Lund
The Attach API is optional for Users running the product. The Attach API is required to compile the classes that use the Attach API and to run tests that cover this feature (such as "--pid"). On Wed, Jul 8, 2020 at 12:11 PM Anthony Baker wrote: > I thought we made the dependency on the Attach

Re: Flaky test caused by missing JDK dependency

2020-07-08 Thread Anthony Baker
I thought we made the dependency on the Attach API optional when we added support for JDK 11? Anthony > On Jul 8, 2020, at 10:17 AM, Kirk Lund wrote: > > To transition away from Attach API, the community needs a proposal to do so > and we'll need to deprecate the GFSH options that depend on

Re: Flaky test caused by missing JDK dependency

2020-07-08 Thread Kirk Lund
To transition away from Attach API, the community needs a proposal to do so and we'll need to deprecate the GFSH options that depend on Attach API such as "--pid" in "status server --pid 20938". Even then we're looking at a minimum of one major release before we can remove options after they are

Re: Flaky test caused by missing JDK dependency

2020-07-08 Thread Sean Goller
The Liberica JDK does not include the Attach API. I'm investigating why. Given the inherent insecurity of this API, I recommend we transition away from using it. From: Kirk Lund Sent: Monday, July 6, 2020 10:36 AM To: dev@geode.apache.org Subject: Flaky test