RFR: 8343936: Adjust timeout in test javax/management/monitor/DerivedGaugeMonitorTest.java

Daniel Fuchs dfuchs at openjdk.org
Mon Nov 11 14:49:12 UTC 2024


On Mon, 11 Nov 2024 11:06:17 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

> Test timeout change to cope with -Xcomp runs.
> Long-standing problemlisted test.
> 
> With -Xcomp locally I could see a failure on the second attempt.
> After this change, ran more runs locally and 40 runs per platform in CI are OK.

Looks reasonable to me. A possible alternative would be to use `jdk.test.lib.Utils.adjustTimeout(long)` to take into account the `timeoutFactor`. IIRC tests run with an icreased `timeoutFactor` when run with -Xcomp

-------------

PR Review: https://git.openjdk.org/jdk/pull/22008#pullrequestreview-2427486147


More information about the serviceability-dev mailing list