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

Update gopsutils to fix #4037 #4045

Merged
merged 2 commits into from
Apr 20, 2018
Merged

Update gopsutils to fix #4037 #4045

merged 2 commits into from
Apr 20, 2018

Conversation

lpic10
Copy link
Contributor

@lpic10 lpic10 commented Apr 19, 2018

Required for all PRs:

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

Fixes #4037

Instead of the latest commit I'm picking the gopsutils commit just after current one used in telegraf, to avoid adding unrelated changes

@danielnelson
Copy link
Contributor

danielnelson commented Apr 20, 2018

We should pull the next immediate change then since it was a fix I did for #3750. Most importantly here we have tested both of the changes so hopefully there will be no surprises.

Can you update to a5c2888e464b14fa882c2a059e0f95716bd45cf1

@danielnelson danielnelson added this to the 1.6.1 milestone Apr 20, 2018
@danielnelson danielnelson added the fix pr to fix corresponding bug label Apr 20, 2018
@danielnelson danielnelson merged commit eacf11f into influxdata:master Apr 20, 2018
danielnelson pushed a commit that referenced this pull request Apr 20, 2018
@W0CHP
Copy link

W0CHP commented Apr 24, 2018

Confirmed fixed on FreeBSD, amd64. Submitted 1.6.1 port patches upstream to FreeBSD-ports for their inclusion. https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=227738

arkady-emelyanov pushed a commit to arkady-emelyanov/telegraf that referenced this pull request May 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.

3 participants