On Tue, 28 Sep 2021 15:54:09 GMT, Aleksey Shipilev <sh...@openjdk.org> wrote:
> Recently added test fails on my desktop in the course of newly added `tier4` > runs. The reason is simple: the test verifies that lsof outputs no more than > LIMIT_LINES (2) lines. And on my machine, lsof prints some warnings that blow > that limit. In the patch, lsof -w is used to suppress the warnings. > > Additional testing: > - [x] Affected test now passes Marked as reviewed by dfuchs (Reviewer). I saw the test failed once on a linux ARM 64 machine - but it didn't fail again after 50 more runs. I also believe that the failure has nothing to do with your changes, so from my perspective your good to go. In case you're interested here is the tail of the log file that shows the failure. I suspect this a one off where the OS hadn't reclaimed the file descriptor within the expected delay (maybe due to other activity occurring on the machine). ... 99 0 1 2 3 4 5 6 7 8 9 end COMMAND PID USER FD TYPE DEVICE SIZE/OFF NODE NAME java 2958962 mach5-one 5u unix 0x0000000000000000 0t0 217247452 type=STREAM java 2958962 mach5-one 8u unix 0x0000000000000000 0t0 217252225 type=STREAM ----------System.err:(13/835)---------- java.lang.RuntimeException: Failed: detected unclosed FD. at CloseDescriptors.main(CloseDescriptors.java:92) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.base/java.lang.reflect.Method.invoke(Method.java:568) at com.sun.javatest.regtest.agent.MainWrapper$MainThread.run(MainWrapper.java:127) at java.base/java.lang.Thread.run(Thread.java:833) JavaTest Message: Test threw exception: java.lang.RuntimeException: Failed: detected unclosed FD. JavaTest Message: shutting down test ------------- PR: https://git.openjdk.java.net/jdk/pull/5739