Skip to content

Output lsof information on suspected leakage#5945

Closed
ahus1 wants to merge 1 commit into
eclipse-vertx:masterfrom
ahus1:is-5944-show-lsof-output-on-test-failure
Closed

Output lsof information on suspected leakage#5945
ahus1 wants to merge 1 commit into
eclipse-vertx:masterfrom
ahus1:is-5944-show-lsof-output-on-test-failure

Conversation

@ahus1

@ahus1 ahus1 commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

Closes #5944

Motivation:

Show lsof information when the test fails to allow an analysis of what files changes.

Example output:

lsof before:
COMMAND    PID     USER   FD   TYPE             DEVICE  SIZE/OFF     NODE NAME
java    791300 aschwart  cwd    DIR               0,41        32 21285744 /home/aschwart/redhat/workspace/vert.x/vertx-core
java    791300 aschwart  rtd    DIR               0,37       266      256 /
java    791300 aschwart  txt    REG               0,41     12984 20643956 /home/aschwart/.jdks/corretto-17.0.17/bin/java
...

lsof after:
COMMAND    PID     USER   FD   TYPE             DEVICE  SIZE/OFF     NODE NAME
java    791300 aschwart  cwd    DIR               0,41        32 21285744 /home/aschwart/redhat/workspace/vert.x/vertx-core
java    791300 aschwart  rtd    DIR               0,37       266      256 /
java    791300 aschwart  txt    REG               0,41     12984 20643956 /home/aschwart/.jdks/corretto-17.0.17/bin/java
...

@ahus1
ahus1 marked this pull request as ready for review February 6, 2026 16:06
@vietj

vietj commented Feb 9, 2026

Copy link
Copy Markdown
Member

how does such failure handling work on windows ?

@ahus1

ahus1 commented Feb 9, 2026

Copy link
Copy Markdown
Contributor Author

@vietj - the detection logic of a leak is still identical. The only change in this PR is to augment the information on failure.

So the behavior of failing/not failing is as before. But when you run on Linux, it will show you the full lsof output and you might be closer to understand why this failed. On Windows and everywhere such a CLI is not available, the output would just be skipped.

@ahus1

ahus1 commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

This has run stale and acquired conflicts. Closing.

@ahus1 ahus1 closed this Jun 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FileDescriptorLeakDetector should output lsof for analysis

2 participants