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

Uptime missing when running in docker container #4037

Closed
lpic10 opened this issue Apr 18, 2018 · 2 comments
Closed

Uptime missing when running in docker container #4037

lpic10 opened this issue Apr 18, 2018 · 2 comments
Labels
bug unexpected problem or unintended behavior upstream bug or issues that rely on dependency fixes
Milestone
1.6.1

Comments

@lpic10
Copy link
Contributor

lpic10 commented Apr 18, 2018

Bug report

System info:

telegraf 1.6 running inside docker (CentOS Linux release 7.3.1611)
host: RHEL 7.4

Telegraf 1.6 is not retrieving the uptime information when running inside the container from some hosts.

Possibly related to issue shirou/gopsutil#507 and recent gops update
@russorat russorat added the bug unexpected problem or unintended behavior label Apr 18, 2018
@danielnelson danielnelson added this to the 1.7.0 milestone Apr 18, 2018
@danielnelson danielnelson added the upstream bug or issues that rely on dependency fixes label Apr 18, 2018
@danielnelson
Copy link
Contributor

Can you check if the upstream bugfix solves the issue? I think this can probably wait for 1.7, which would be safer than pulling in a new release of gopsutil.

@lpic10 lpic10 mentioned this issue Apr 19, 2018
Merged
3 tasks
@lpic10
Copy link
Contributor Author

lpic10 commented Apr 19, 2018

Yes, it seems to do the trick.
As telegraf is using gopsutils just one commit behind this one, I've opened a PR to update to it. So should be safe to apply to 1.6

@danielnelson danielnelson modified the milestones: 1.7.0, 1.6.1 Apr 20, 2018
danielnelson pushed a commit that referenced this issue Apr 20, 2018
@lpic10 @danielnelson
Update gopsutils to include fixes for #4037 and #3750 (#4045)
eacf11f
danielnelson pushed a commit that referenced this issue Apr 20, 2018
@lpic10 @danielnelson
Update gopsutils to include fixes for #4037 and #3750 (#4045)
291a8f0 
(cherry picked from commit eacf11f)
arkady-emelyanov pushed a commit to arkady-emelyanov/telegraf that referenced this issue May 18, 2018
@lpic10 @arkady-emelyanov
Update gopsutils to include fixes for influxdata#4037 and influxdata#…
0840c7a 
…3750 (influxdata#4045)
maxunt pushed a commit that referenced this issue Jun 26, 2018
@lpic10 @danielnelson
Update gopsutils to include fixes for #4037 and #3750 (#4045)
4df09fb
otherpirate pushed a commit to otherpirate/telegraf that referenced this issue Mar 15, 2019
@lpic10 @otherpirate
Update gopsutils to include fixes for influxdata#4037 and influxdata#…
9fc4aff 
…3750 (influxdata#4045)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug unexpected problem or unintended behavior upstream bug or issues that rely on dependency fixes
Projects
None yet
Milestone
1.6.1
Development

No branches or pull requests
3 participants
@danielnelson @russorat @lpic10