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