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

Report available fields if utmp is unreadable #4043

Merged
merged 1 commit into from
Apr 18, 2018

Conversation

danielnelson
Copy link
Contributor

closes #3837

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

@danielnelson danielnelson added the fix pr to fix corresponding bug label Apr 18, 2018
@danielnelson danielnelson added this to the 1.6.1 milestone Apr 18, 2018
@danielnelson danielnelson force-pushed the skip-users-if-utmp-unreadable branch from ad3eb62 to 31ed2b4 Compare April 18, 2018 23:28
@danielnelson
Copy link
Contributor Author

@Crapworks This change ignores permission failed errors reading utmp, and will skip reporting the field. For now I will leave it to the admin to add the user to the utmp group if they would like to get the n_users field.

@danielnelson danielnelson merged commit 120be7e into master Apr 18, 2018
@danielnelson danielnelson deleted the skip-users-if-utmp-unreadable branch April 18, 2018 23:55
danielnelson added a commit that referenced this pull request Apr 18, 2018
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix pr to fix corresponding bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[[inputs.system]] fails on SuSE with permission denied for utmp
1 participant