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

[metricbeat] Fix errors in process summary on latest Linux kernels #6306

Merged
merged 1 commit into from
Feb 12, 2018

Conversation

kesslerm
Copy link
Contributor

@kesslerm kesslerm commented Feb 7, 2018

On Linux 4.14 and 4.15, metricbeat logs large numbers of errors such as

2018/02/07 11:19:46.284926 process_summary.go:79: ERR Unknown state <73> for process with pid 4

All affected processes appear to be kernel threads.

<73> translates into I, which is a new status code introduced into the
/proc/<pid>/stat output in kernel commit
torvalds/linux@06eb618

For backwards compatibility, the old and new codes are now used to increment
the count of idle processes reported in the summary.

Fixes #6305.

@ph ph added needs CLA User must sign the Elastic Contributor License before review. and removed needs CLA User must sign the Elastic Contributor License before review. labels Feb 7, 2018
@ph
Copy link
Contributor

ph commented Feb 7, 2018

@karmi This PR should pass the CLA check, @kesslerm is part of the elastic org?

@ph ph added Metricbeat Metricbeat bug labels Feb 7, 2018
@ph
Copy link
Contributor

ph commented Feb 7, 2018

@kesslerm Would you mind adding a changelog entry for metricbeat?

@karmi
Copy link

karmi commented Feb 7, 2018

@ph, there might be a problem with the email address — @kesslerm, I think you've used a different email in the Git commit than is attached to your Github profile. If you add it there, it should work, that's the simplest way of fixing the check.

@ph
Copy link
Contributor

ph commented Feb 7, 2018

@karmi Ah! the email is not linked to his github! gotcha! thanks.

@ruflin ruflin added the module label Feb 8, 2018
@kesslerm kesslerm force-pushed the fix/6305 branch 2 times, most recently from ea05f99 to fda6ffd Compare February 9, 2018 12:10
@kesslerm
Copy link
Contributor Author

kesslerm commented Feb 9, 2018

@ph, @karmi, I've adjusted the commit email address and added a changelog entry before force-updating my branch. Please let me know if this is OK now.

On Linux 4.14 and 4.15, metricbeat logs large numbers of errors such as

```
2018/02/07 11:19:46.284926 process_summary.go:79: ERR Unknown state <73> for process with pid 4
```

All affected processes appear to be kernel threads.

`<73>` translates into `I`, which is a new status code introduced into the
`/proc/<pid>/stat` output in kernel commit
torvalds/linux@06eb618

For backwards compatibility, the old and new codes are now used to increment
the count of idle processes reported in the summary.
Copy link
Contributor

@kvch kvch left a comment

Choose a reason for hiding this comment

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

WFG

@ph ph added the needs_backport PR is waiting to be backported to other branches. label Feb 9, 2018
@ruflin ruflin merged commit b81fbd4 into elastic:master Feb 12, 2018
adriansr pushed a commit to adriansr/beats that referenced this pull request May 22, 2018
…lastic#6306)

On Linux 4.14 and 4.15, metricbeat logs large numbers of errors such as

```
2018/02/07 11:19:46.284926 process_summary.go:79: ERR Unknown state <73> for process with pid 4
```

All affected processes appear to be kernel threads.

`<73>` translates into `I`, which is a new status code introduced into the
`/proc/<pid>/stat` output in kernel commit
torvalds/linux@06eb618

For backwards compatibility, the old and new codes are now used to increment
the count of idle processes reported in the summary.

(cherry picked from commit b81fbd4)
@adriansr adriansr removed the needs_backport PR is waiting to be backported to other branches. label May 22, 2018
adriansr pushed a commit to adriansr/beats that referenced this pull request May 22, 2018
…lastic#6306)

On Linux 4.14 and 4.15, metricbeat logs large numbers of errors such as

```
2018/02/07 11:19:46.284926 process_summary.go:79: ERR Unknown state <73> for process with pid 4
```

All affected processes appear to be kernel threads.

`<73>` translates into `I`, which is a new status code introduced into the
`/proc/<pid>/stat` output in kernel commit
torvalds/linux@06eb618

For backwards compatibility, the old and new codes are now used to increment
the count of idle processes reported in the summary.

(cherry picked from commit b81fbd4)
andrewkroh pushed a commit that referenced this pull request May 31, 2018
…6306) (#7155)

On Linux 4.14 and 4.15, metricbeat logs large numbers of errors such as

```
2018/02/07 11:19:46.284926 process_summary.go:79: ERR Unknown state <73> for process with pid 4
```

All affected processes appear to be kernel threads.

`<73>` translates into `I`, which is a new status code introduced into the
`/proc/<pid>/stat` output in kernel commit
torvalds/linux@06eb618

For backwards compatibility, the old and new codes are now used to increment
the count of idle processes reported in the summary.

(cherry picked from commit b81fbd4)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…lastic#6306) (elastic#7155)

On Linux 4.14 and 4.15, metricbeat logs large numbers of errors such as

```
2018/02/07 11:19:46.284926 process_summary.go:79: ERR Unknown state <73> for process with pid 4
```

All affected processes appear to be kernel threads.

`<73>` translates into `I`, which is a new status code introduced into the
`/proc/<pid>/stat` output in kernel commit
torvalds/linux@06eb618

For backwards compatibility, the old and new codes are now used to increment
the count of idle processes reported in the summary.

(cherry picked from commit f416d5b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants