Bugzilla – Full Text Bug Listing
|
Description
Jay Arthanareeswaran
Any chance of a resolution here? This is blocking our RC2 work. New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/177135 Sorry, no resolution yet. In a minimal build environment (centos-7) lsof can be found: https://ci.eclipse.org/jdt/job/webmaster-test/1/console Can you provide a minimal test case that reproduces the error? New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/176825 Recent test failure https://ci.eclipse.org/jdt/job/eclipse.jdt.core-Gerrit/4152/testReport/junit/org.eclipse.jdt.core.tests.builder/BuilderTests/testLeaksOnCleanBuild/ reads: junit.framework.AssertionFailedError: Failed to read opened file descriptors at org.eclipse.jdt.core.tests.builder.LeakTestsBefore9.testLeaksOnCleanBuild(LeakTestsBefore9.java:36) This indicates the the lsof invocation returned an empty string. @Andrey: in bug 543506 you made the call with this command line "lsof -F n -a -p " + pid + " / -b " On my ubuntu system this causes: lsof: status error on -b: No such file or directory Reason: there doesn't seem to be syntax for '/' in the given way. If I remove that char, no error occurs. What is '/' intended to do? (In reply to Stephan Herrmann from comment #5) > What is '/' intended to do? It was supposed to be the root of the file system. That was used to work on RHEL 7.4, I can check it again. (In reply to Andrey Loskutov from comment #6) > (In reply to Stephan Herrmann from comment #5) > > What is '/' intended to do? > > It was supposed to be the root of the file system. That was used to work on > RHEL 7.4, I can check it again. Then it should probably be the last argument after -b ? the "names" in: lsof [ -?abChlnNOPRtUvVX ] [ -A A ] [ -c c ] [ +c c ] [ +|-d d ] [ +|-D D ] [ +|-e s ] [ +|-E ] [ +|-f [cfgGn] ] [ -F [f] ] [ -g [s] ] [ -i [i] ] [ -k k ] [ -K k ] [ +|-L [l] ] [ +|-m m ] [ +|-M ] [ -o [o] ] [ -p s ] [ +|-r [t[m<fmt>]] ] [ -s [p:s] ] [ -S [t] ] [ -T [t] ] [ -u s ] [ +|-w ] [ -x [fl] ] [ -z [z] ] [ -Z [Z] ] [ -- ] [names] right? Gerrit change https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/176825 was merged to [BETA_JAVA16]. Commit: http://git.eclipse.org/c/jdt/eclipse.jdt.core.git/commit/?id=d2775025c44cae53dc1e67d02b419c4e6c80a72d (In reply to Andrey Loskutov from comment #6) > (In reply to Stephan Herrmann from comment #5) > > What is '/' intended to do? > > It was supposed to be the root of the file system. That was used to work on > RHEL 7.4, I can check it again. I must correct me: this was used to select only regular (physical) files/directories. Something like that was mentioned here: https://serverfault.com/questions/106398/lsof-restrict-output-to-physical-files-only-how I guess that was some undocumented hack that is may be "fixed" in latest lsof versions, and still there on RHEL 7.4 where I'm working on. So if that is not working on gerrit anymore, we can remove "/". Can this one be closed or moved to JDT? Closing as as INVALID as it is not an issue with Jenkins. Feel free to reopen/move if needed. (In reply to Mikaël Barbero from comment #11) > Closing as as INVALID as it is not an issue with Jenkins. Feel free to > reopen/move if needed. Over to JDT New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/178002 (In reply to Andrey Loskutov from comment #9) > So if that is not working on gerrit anymore, we can remove "/". "we" is a dangerous word in team organization, because that guy rarely creates any commits whatsoever. Ever heard of diffusion of responsibility? ;p (In reply to Eclipse Genie from comment #13) > New Gerrit change created: > https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/178002 Trying to prevent the death of relevant tests. *** Bug 562681 has been marked as a duplicate of this bug. *** (In reply to Stephan Herrmann from comment #14) > (In reply to Andrey Loskutov from comment #9) > > So if that is not working on gerrit anymore, we can remove "/". > > "we" is a dangerous word in team organization, because that guy rarely > creates any commits whatsoever. Ever heard of diffusion of responsibility? ;p > > (In reply to Eclipse Genie from comment #13) > > New Gerrit change created: > > https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/178002 > > Trying to prevent the death of relevant tests. Thanks Stephan, I'm just not able to find free time for anything, too busy with my day work. Gerrit change https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/178002 was merged to [master]. Commit: http://git.eclipse.org/c/jdt/eclipse.jdt.core.git/commit/?id=2af7f7fe05757222d516240b39d11bc70398f1d3 (In reply to Eclipse Genie from comment #17) > Gerrit change https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/178002 was > merged to [master]. > Commit: > http://git.eclipse.org/c/jdt/eclipse.jdt.core.git/commit/ > ?id=2af7f7fe05757222d516240b39d11bc70398f1d3 This should do. Unfortunately this causes DNF in jdt.core.tests.builder on Linux https://download.eclipse.org/eclipse/downloads/drops4/I20210323-1240/testResults.php https://download.eclipse.org/eclipse/downloads/drops4/I20210323-1240/testresults/ep420I-unit-cen64-gtk3-java11_linux.gtk.x86_64_11/org.eclipse.jdt.core.tests.builder.BuilderTests.txt java.lang.Exception: ThreadDump for thread "DumpStackTracesTimer" at java.base@11.0.10/java.lang.Thread.dumpThreads(Native Method) at java.base@11.0.10/java.lang.Thread.getAllStackTraces(Thread.java:1658) at org.eclipse.ui.testing.dumps.TimeoutDumpTimer.dumpStackTraces(TimeoutDumpTimer.java:238) at org.eclipse.ui.testing.dumps.TimeoutDumpTimer.logStackTraces(TimeoutDumpTimer.java:224) at org.eclipse.ui.testing.dumps.TimeoutDumpTimer.dump(TimeoutDumpTimer.java:161) at org.eclipse.ui.testing.dumps.TimeoutDumpTimer.run(TimeoutDumpTimer.java:138) at java.base@11.0.10/java.util.TimerThread.mainLoop(Timer.java:556) at java.base@11.0.10/java.util.TimerThread.run(Timer.java:506) java.lang.Exception: ThreadDump for thread "Worker-1" at java.base@11.0.10/java.lang.Object.wait(Native Method) at org.eclipse.core.internal.jobs.WorkerPool.sleep(WorkerPool.java:200) at org.eclipse.core.internal.jobs.WorkerPool.startJob(WorkerPool.java:242) at org.eclipse.core.internal.jobs.Worker.run(Worker.java:58) java.lang.Exception: ThreadDump for thread "main" at java.base@11.0.10/java.io.FileInputStream.readBytes(Native Method) at java.base@11.0.10/java.io.FileInputStream.read(FileInputStream.java:279) at java.base@11.0.10/java.io.BufferedInputStream.read1(BufferedInputStream.java:290) at java.base@11.0.10/java.io.BufferedInputStream.read(BufferedInputStream.java:351) at java.base@11.0.10/sun.nio.cs.StreamDecoder.readBytes(StreamDecoder.java:284) at java.base@11.0.10/sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:326) at java.base@11.0.10/sun.nio.cs.StreamDecoder.read(StreamDecoder.java:178) at java.base@11.0.10/java.io.InputStreamReader.read(InputStreamReader.java:181) at java.base@11.0.10/java.io.BufferedReader.fill(BufferedReader.java:161) at java.base@11.0.10/java.io.BufferedReader.readLine(BufferedReader.java:326) at java.base@11.0.10/java.io.BufferedReader.readLine(BufferedReader.java:392) at org.eclipse.jdt.core.tests.builder.AbstractLeakTest.readLsofLines(AbstractLeakTest.java:191) at org.eclipse.jdt.core.tests.builder.AbstractLeakTest.getOpenDescriptors(AbstractLeakTest.java:174) at org.eclipse.jdt.core.tests.builder.AbstractLeakTest.checkOpenDescriptors(AbstractLeakTest.java:160) at org.eclipse.jdt.core.tests.builder.AbstractLeakTest.assertNotLeaked(AbstractLeakTest.java:135) at org.eclipse.jdt.core.tests.builder.AbstractLeakTest.internalTestUsedLibraryLeaks(AbstractLeakTest.java:83) at org.eclipse.jdt.core.tests.builder.AbstractLeakTest.testLeaksOnCleanBuild(AbstractLeakTest.java:61) at org.eclipse.jdt.core.tests.builder.LeakTestsBefore9.testLeaksOnCleanBuild(LeakTestsBefore9.java:36) at java.base@11.0.10/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) New Gerrit change created: https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/178320 Gerrit change https://git.eclipse.org/r/c/jdt/eclipse.jdt.core/+/178320 was merged to [master]. Commit: http://git.eclipse.org/c/jdt/eclipse.jdt.core.git/commit/?id=09bb69aa67a0339779e60595227112c004cfd25d Works in I20210324-1800 verified based on last comment. |