> 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: remove bug id ------------- Changes: - all: https://git.openjdk.org/jdk/pull/28595/files - new: https://git.openjdk.org/jdk/pull/28595/files/58cb9d59..f0cef51b Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=28595&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28595&range=01-02 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 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
