Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Probe: Don't report dead or defunct processes #3379

Merged
merged 1 commit into from
Oct 16, 2018

Conversation

bboreham
Copy link
Collaborator

@bboreham bboreham commented Oct 9, 2018

If the state is reported as dead or zombie, return an error so the process will be skipped.

This will fix the direct cause of the situation described at #3330

If the state is reported as dead or zombie, return an error so the
process will be skipped.
@bboreham bboreham changed the title Don't report dead or defunct processes Probe: Don't report dead or defunct processes Oct 13, 2018
Copy link
Contributor

@satyamz satyamz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried this with sample zombie process. It worked!

$ ./zombie & 
[1] 4148
$ parent process - 4148
child process - 4149
$ ps axl  | grep Z
F   UID   PID  PPID PRI  NI    VSZ   RSS WCHAN  STAT TTY        TIME COMMAND
1  1001  4149  4148  20   0      0     0 -      Z    pts/0      0:00 [zombie] <defunct>
4  1001  4321 30988  20   0   9120   448 -      S+   pts/0      0:00 grep Z

screenshot from 2018-10-16 12-02-55
Process was not in the Unconnected nodes too.

@bboreham bboreham merged commit 2a5f871 into master Oct 16, 2018
@bboreham bboreham deleted the dont-report-dead-processes branch September 13, 2019 15:33
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.

None yet

2 participants