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