Hi Daniil,
Thanks for cleaning this up. I think this should be fixed under
JDK-8242009. JDK-8238561 involves more than just this one issue.
Is there a reason why you didn't just change JDKToolLauncher to have an
option or API to add the args?
Why are you calling Utils.addTestJavaOpts() instead of
Utils.getTestJavaOpts()?
Is your change causing -Xshowversion to be passed? Do you know where it
is coming from?
thanks,
Chris
[1] https://bugs.openjdk.java.net/browse/JDK-8242009
On 4/22/20 10:48 AM, Daniil Titov wrote:
Please review the change [1] that ensures that VM and test options are
forwarded to
j*-tools when they are launched from serviceability/sa tests.
In particular, it will ensure that passed to the tests maximum heap size
settings ( -XX:MaxRAMPercentage)
are also honored by j*-tools serviceability/sa tests launch.
The tests that expect an empty output were corrected to ignore the product
version printed
in the error stream since in some tiers the tests are run with ' -showversion'
VM option.
Testing: Mach5 tests for tier1 - tier7 passed.
[1] http://cr.openjdk.java.net/~dtitov/8238561/webrev.01
[2] https://bugs.openjdk.java.net/browse/JDK-8238561
Thank you,
Daniil