RFR: 7904088: Include the timestamp to the log message when a timeout is fired
Jaikiran Pai
jpai at openjdk.org
Fri Sep 26 09:22:51 UTC 2025
On Fri, 26 Sep 2025 07:01:36 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
> Can I please get a review of this trivial change which includes the timestamp for a log message when a test action times out?
>
> Before this change, jtreg would log a message like:
>
>> Timeout signalled after 120 seconds
>
> Now it will also include a timestamp for that log message. This should help during investigations of timing out tests:
>
>> [08:10:07.344] Timeout signalled after 120 seconds
>
> The change is a one liner in `src/share/classes/com/sun/javatest/regtest/agent/Alarm.java`. Rest of the changes are mere refactoring to allow for using the same `DateTimeFormat` consistently in a couple of other places.
Thank you Christian for the review.
-------------
PR Comment: https://git.openjdk.org/jtreg/pull/293#issuecomment-3337572476
More information about the jtreg-dev
mailing list