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

Temperature input plugin stops working when WiFi is turned off #5692

Closed
marcv81 opened this issue Apr 8, 2019 · 3 comments · Fixed by #5941
Closed

Temperature input plugin stops working when WiFi is turned off #5692

marcv81 opened this issue Apr 8, 2019 · 3 comments · Fixed by #5941
Labels
bug unexpected problem or unintended behavior upstream bug or issues that rely on dependency fixes
Milestone

Comments

@marcv81
Copy link
Contributor

marcv81 commented Apr 8, 2019

Relevant telegraf.conf:

[[inputs.temp]]

System info:

  • Telegraf 1.10.1-1 installed from .deb package / Ubuntu 16.04 + Kernel 5.0
  • Intel Wireless LAN hardware, iwlwifi module

Steps to reproduce:

  1. Enable WiFi, connect to a network
  2. Enable the inputs.temp plugin
  3. Observe that the plugin is working correctly: various temp metrics should exist, including one called temp,sensor=iwlwifi_input
  4. Disconnect from the network, turn off WiFi

Expected behavior:

No new metrics should be produced for temp,sensor=iwlwifi_input. New metrics should still be produced for everything else in temp.

Actual behavior:

No new metric is produced for anything in temp.

Additional info:

Apr 08 14:36:50 telegraf[6305]: 2019-04-08T06:36:50Z E! [inputs.temp]: Error in plugin: error getting temperatures info: read /sys/class/hwmon/hwmon4/temp1_input: no data available
@glinton
Copy link
Contributor

glinton commented Apr 8, 2019

We could add to the check here to return only if len(temps) == 0, but you could still potentially see partial or missing data. This is due to the upstream gopsutil library returning on file read errors.

This seems like an upstream issue, as implementing the len workaround would create a new issue for incomplete data.

@glinton glinton added bug unexpected problem or unintended behavior upstream bug or issues that rely on dependency fixes labels Apr 8, 2019
@danielnelson
Copy link
Contributor

@glinton Could you create a new issue on the gopsutil project and link to it here. I agree the upstream code should probably skip errors.

@marcv81
Copy link
Contributor Author

marcv81 commented Jun 1, 2019

Related PR: #5941.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unexpected problem or unintended behavior upstream bug or issues that rely on dependency fixes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants