RFR: 7904104: intermittent test failure in TimeoutDefaultSecondsWaiter

Vladimir Petko vpetko at openjdk.org
Tue Oct 28 04:18:08 UTC 2025


`victim.interrupt()` is not needed if the process exits  after the kill signal. 
This PR only calls interrupt() if the process fails to exit or the wait is interrupted for some reason. 

This allows TimeoutDefaultSecondsWaiter tests to pass. 

Alternative: update assertion in TimeoutDefaultSecondsWaiter test.

---------
### Progress
- [ ] Change must be properly reviewed (1 review required, with at least 1 [Reviewer](https://openjdk.org/bylaws#reviewer))
- [x] Change must not contain extraneous whitespace
- [x] Commit message must refer to an issue

### Error
 ⚠️ The pull request body must not be empty.



### Reviewing
<details><summary>Using <code>git</code></summary>

Checkout this PR locally: \
`$ git fetch https://git.openjdk.org/jtreg.git pull/296/head:pull/296` \
`$ git checkout pull/296`

Update a local copy of the PR: \
`$ git checkout pull/296` \
`$ git pull https://git.openjdk.org/jtreg.git pull/296/head`

</details>
<details><summary>Using Skara CLI tools</summary>

Checkout this PR locally: \
`$ git pr checkout 296`

View PR using the GUI difftool: \
`$ git pr show -t 296`

</details>
<details><summary>Using diff file</summary>

Download this PR as a diff file: \
<a href="https://git.openjdk.org/jtreg/pull/296.diff">https://git.openjdk.org/jtreg/pull/296.diff</a>

</details>

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

Commit messages:
 - chore: remove whitespace
 - fix: interrupt the main thread if the process fails to exit
 - fix: do not interrupt the main thread

Changes: https://git.openjdk.org/jtreg/pull/296/files
  Webrev: https://webrevs.openjdk.org/?repo=jtreg&pr=296&range=00
  Issue: https://bugs.openjdk.org/browse/CODETOOLS-7904104
  Stats: 11 lines in 1 file changed: 8 ins; 0 del; 3 mod
  Patch: https://git.openjdk.org/jtreg/pull/296.diff
  Fetch: git fetch https://git.openjdk.org/jtreg.git pull/296/head:pull/296

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


More information about the jtreg-dev mailing list