Integrated: 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.

This pull request has now been integrated.

Changeset: ea710f96
Author:    Jaikiran Pai <jpai at openjdk.org>
URL:       https://git.openjdk.org/jtreg/commit/ea710f961383c0ce27a75ac6b9ddc802b3922142
Stats:     67 lines in 5 files changed: 48 ins; 15 del; 4 mod

7904088: Include the timestamp to the log message when a timeout is fired

Reviewed-by: cstein

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

PR: https://git.openjdk.org/jtreg/pull/293


More information about the jtreg-dev mailing list