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

Jaikiran Pai jpai at openjdk.org
Mon Aug 19 04:30:20 UTC 2024


> 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

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

Changes:
  - all: https://git.openjdk.org/jtreg/pull/215/files
  - new: https://git.openjdk.org/jtreg/pull/215/files/7404620f..6130232d

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jtreg&pr=215&range=02
 - incr: https://webrevs.openjdk.org/?repo=jtreg&pr=215&range=01-02

  Stats: 5 lines in 2 files changed: 0 ins; 0 del; 5 mod
  Patch: https://git.openjdk.org/jtreg/pull/215.diff
  Fetch: git fetch https://git.openjdk.org/jtreg.git pull/215/head:pull/215

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


More information about the jtreg-dev mailing list