[crac] RFR: 8362837: [CRaC] jdk/crac/MXBean.java can fail on macOS [v2]
Timofei Pushkin
tpushkin at openjdk.org
Sun Jul 20 19:56:19 UTC 2025
On Sun, 20 Jul 2025 19:46:48 GMT, Jan Kratochvil <jkratochvil at openjdk.org> wrote:
>> test/jdk/jdk/crac/MXBean.java line 90:
>>
>>> 88: long restoreUptime = Long.parseLong(output.firstMatch("UptimeSinceRestore ([0-9-]+)", 1));
>>> 89: System.err.println("restoreUptime=" + restoreUptime);
>>> 90: if (restoreUptime < 0) {
>>
>> AFAIK we have not had problems with this check, only with the below one. "UptimeSinceRestore" should be close to 0 on all platforms
>
> What do you exactly mean by `close to 0`? I already mention here in the initial comment I did not implement `close to 0`. Do you mean less than `TIME_TOLERANCE`? But that is exactly the problem why it sometimes fails on Mac CI.
>
> I am going to remove the `restoreUptime < 0` test when you do not like it.
CI was failing for the check below AFAIK. In my understanding this one should not be failing the way it is.
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/246#discussion_r2217936481
More information about the crac-dev
mailing list