RFR: 7903781: Report the process id of the agent or other VM that was used for a jtreg action [v3]

[Jonathan Gibbons]

On Mon, 19 Aug 2024 04:30:20 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Can I please get a review of this change which proposes to repor the process id of the process that was used to run a jtreg action? This change addresses https://bugs.openjdk.org/browse/CODETOOLS-7903781.
>> 
>> As noted in that linked enhancement request, this change will report the process id  the `section` messages of each action. This will now look like:
>> 
>> 
>> #section:compile
>> ----------messages:(7/297)----------
>> command: compile test/streams/FileDescriptorTest.java
>> reason: .class file out of date or does not exist
>> started: Fri Jul 26 00:56:01 UTC 2024
>> Mode: othervm
>> Process id: 88469
>> finished: Fri Jul 26 00:56:01 UTC 2024
>> elapsed time (seconds): 0.254
>> 
>> ....
>> 
>> #section:main
>> ----------messages:(7/232)----------
>> command: main FileDescriptorTest
>> reason: User specified action: run main FileDescriptorTest 
>> started: Fri Jul 26 00:56:01 UTC 2024
>> Mode: othervm
>> Process id: 88470
>> finished: Fri Jul 26 00:56:01 UTC 2024
>> elapsed time (seconds): 0.042
>> 
>> Notice the new "Process id: ..." line in those sections.
>
> Jaikiran Pai has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - rename destroyForciblyMethod to DESTROY_FORCIBLY_METHOD
>  - Jon's suggestion for javadoc text

Marked as reviewed by jjg (Lead).

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

PR Review: https://git.openjdk.org/jtreg/pull/215#pullrequestreview-2255411316