Hi

The fix looks good itself (you still need 'R'eview). However I see about 20 other SA/ tests where lingeredApp is still executed with vm.opts only. Even it doesn't cause any failures it means that tests silently ignore java.opts set by user.

I think it makes sense to fix them also or file new issue for this activity.

Leonid

On 1/14/20 11:38 AM, coleen.phillim...@oracle.com wrote:
Summary: Use getTestJavaOpts() instead of getVmOptions() because of mach5 configuration settings.

Tested with hs-tier3 with no failures anymore.

open webrev at http://cr.openjdk.java.net/~coleenp/2020/8236917.01/webrev
bug link https://bugs.openjdk.java.net/browse/JDK-8236917

Thanks,
Coleen

Reply via email to