On Fri, 6 Oct 2023 19:10:50 GMT, Kevin Walls <kev...@openjdk.org> wrote:
> From studying test failures, it looks like the way the test identifies its > related processes is failing. > It checks the mainArgs of a process by attaching, and looks like it > occasionally misses getting a valid match. The hasMainArgs method ignores > exceptions as it is expecting some exceptions: it is going to test unrelated > java process which happen to start. > > It should retry this main args check on failure, but not too many times to be > a burden on other valid unrelated processes, and should also log the PIDs > that have an issue so we can see if this is part of any future failure. > > Other small logging changes so we can see more easily the progress through > the test. This pull request has now been integrated. Changeset: a045258a Author: Kevin Walls <kev...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/a045258ae2eb02daa17a9a9799a666f42daa7e20 Stats: 51 lines in 1 file changed: 33 ins; 10 del; 8 mod 8209595: MonitorVmStartTerminate.java timed out Reviewed-by: sspitsyn, cjplummer, lmesnik ------------- PR: https://git.openjdk.org/jdk/pull/16077
