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.

Sign up for GitHub

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

Jump to bottom

Check if SNMP6 directory exists (#5403) #5413

Merged
merged 2 commits into from
Feb 13, 2019
Merged

Check if SNMP6 directory exists (#5403) #5413

merged 2 commits into from
Feb 13, 2019

Conversation

deric
Copy link
Contributor

@deric deric commented Feb 12, 2019 •
edited
Loading

Silently ignore missing /proc/stat/snmp6 directory

Required for all PRs:

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

Resolves #5403

glinton
glinton approved these changes Feb 12, 2019
View reviewed changes
Copy link
Contributor

@glinton glinton left a comment

Choose a reason for hiding this comment

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

Thanks

@glinton glinton added this to the 1.10.0 milestone Feb 12, 2019
@glinton glinton added the fix pr to fix corresponding bug label Feb 12, 2019
@danielnelson danielnelson merged commit ee5827c into influxdata:master Feb 13, 2019
trevorwhitney pushed a commit to trevorwhitney/telegraf that referenced this pull request Feb 14, 2019
@deric
Remove error log when snmp6 directory does not exists with nstat input (
1af97cc 
influxdata#5403) (influxdata#5413)
@danielnelson danielnelson modified the milestones: 1.10.0, 1.9.5 Feb 19, 2019
danielnelson pushed a commit that referenced this pull request Feb 19, 2019
@deric @danielnelson
Remove error log when snmp6 directory does not exists with nstat input (
db200bd 
#5403) (#5413)

(cherry picked from commit ee5827c)
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
@deric @otherpirate
Remove error log when snmp6 directory does not exists with nstat input (
f9337b6 
influxdata#5403) (influxdata#5413)
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
@deric @otherpirate
Remove error log when snmp6 directory does not exists with nstat input (
40ca350 
influxdata#5403) (influxdata#5413)
dupondje pushed a commit to dupondje/telegraf that referenced this pull request Apr 22, 2019
@deric
Remove error log when snmp6 directory does not exists with nstat input (
0ceee00 
influxdata#5403) (influxdata#5413)
bitcharmer pushed a commit to bitcharmer/telegraf that referenced this pull request Oct 18, 2019
@deric @bitcharmer
Remove error log when snmp6 directory does not exists with nstat input (
660e3f2 
influxdata#5403) (influxdata#5413)
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
@deric
Remove error log when snmp6 directory does not exists with nstat input (
59e058a 
influxdata#5403) (influxdata#5413)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielnelson danielnelson danielnelson left review comments

@glinton glinton glinton approved these changes

Assignees
No one assigned
Labels
fix pr to fix corresponding bug
Projects
None yet
Milestone
1.9.5
Development

Successfully merging this pull request may close these issues.
3 participants
@deric @danielnelson @glinton