> Can I please get a review of this test-only change which improves the > debuggability of the `java/rmi/server/RemoteServer/AddrInUse.java` test? > > As noted in https://bugs.openjdk.org/browse/JDK-8213699, this test fails > intermittently. The test code launches a Thread which does a > `LocateRegistry.createRegistry(port)`. The test then expects that call to > return within (an arbitrary) 10 seconds and if it doesn't, then it considers > that the test has ended up reproducing a bug which would cause a hang in the > implementation of `LocateRegistry.createRegistry(...)` method. > > The 10 seconds is a reasonable timeout, I think even for busy hosts. But we > have seen this test fail because the launched thread which does the > `LocateRegistry.createRegistry(...)` has either not started or completed > within that period. > > The changes in this PR updates that test code to remove the arbitrary 10 > second timeout and instead just wait for the launched thread to complete. If > the test doesn't complete within the configured jtreg test timeout (which by > default is 2 minutes), then the jtreg and its failure handler infrastructure > will gather the necessary thread dump and other states to help debug why the > test timed out. This should help understand such intermittent failures in > future (if it continues to fail). > > I have triggered a tier testing of this change in our CI and will run a test > repeat too.
Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision: rename failure to registryExportFailure ------------- Changes: - all: https://git.openjdk.org/jdk/pull/28595/files - new: https://git.openjdk.org/jdk/pull/28595/files/71418e31..58cb9d59 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=28595&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28595&range=00-01 Stats: 7 lines in 1 file changed: 0 ins; 1 del; 6 mod Patch: https://git.openjdk.org/jdk/pull/28595.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/28595/head:pull/28595 PR: https://git.openjdk.org/jdk/pull/28595
