hddtemp sensors not updating correctly #5904

Closed
tsumaru720 opened this Issue Feb 15, 2017 · 8 comments

Comments

Projects
None yet
4 participants
@tsumaru720
Contributor

tsumaru720 commented Feb 15, 2017

Hi Fellas,

I noticed that last Thursday, graphs for my hddtemps are not displaying correctly in LibreNMS

https://s3-eu-west-1.amazonaws.com/uploads-eu.hipchat.com/23738/290850/cNy6rMcoRcNJD1h/libre.png
https://s3-eu-west-1.amazonaws.com/uploads-eu.hipchat.com/23738/290850/uS0my0ZLfJXWpFH/upload.png

My daily.sh cron runs at 00:15 every day which is about the time this seems to have occured (as per the graph)

As per request from @laf on IRC - Output of (snipped ofc):

./poller.php -r -f -m applications,unix-agent -h HOSTNAME -d

http://pastebin.com/y7sCfhK7

Permissions appear to be correct (and the files are being updated, by rrdcache) however it seems zero value is being inserted into the RRDs themselves - no other issues with RRD files as far as I can tell

# ls -alctr | grep hddtemp-4
-rw-rw-r--  1 librenms librenms  125000 Feb 15 11:05 sensor-temperature-hddtemp-4.rrd

# rrdtool fetch sensor-temperature-hddtemp-4.rrd AVERAGE -s 1486598400 -e 1486602000
                         sensor

1486598700: 3.0000000000e+01
1486599000: 3.0000000000e+01
1486599300: 1.2000000000e+00
1486599600: 0.0000000000e+00
1486599900: 0.0000000000e+00
1486600200: 0.0000000000e+00
1486600500: 0.0000000000e+00
1486600800: 0.0000000000e+00
1486601100: 0.0000000000e+00
1486601400: 0.0000000000e+00
1486601700: 0.0000000000e+00
1486602000: 0.0000000000e+00
1486602300: 0.0000000000e+00

As evident by the paste, the temps seem to be exposed correctly to LibreNMS

  • Is your install up to date? within last 24 hours
# date
Wed Feb 15 11:01:49 GMT 2017

[WARN]  Your install is out of date, last update: Wed, 15 Feb 2017 00:06:00 +0000
  • Please include all of the information between the ==================================== section of ./validate.php which you can run from the cli.
Component Version
LibreNMS 49b5844
DB Schema 164
PHP 5.5.9-1ubuntu4.21
MySQL 5.5.53-0ubuntu0.14.04.1
RRDTool 1.4.7
SNMP NET-SNMP 5.7.2
  • Please provide ALL info asked for here.

n/a

  • Please provide as much detail as possible.
  • Please do NOT post more than 10 lines of debug information here, use a pastebin service or GitHub Gists.
@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 15, 2017

Member

Can you also provide the output of ./poller.php -h HOSTNAME -d -r -f -m sensors

Member

laf commented Feb 15, 2017

Can you also provide the output of ./poller.php -h HOSTNAME -d -r -f -m sensors

@laf laf added the Needs-Info label Feb 15, 2017

@tsumaru720

This comment has been minimized.

Show comment
Hide comment
@tsumaru720

This comment has been minimized.

Show comment
Hide comment
@tsumaru720

tsumaru720 Feb 16, 2017

Contributor

@laf Looks to still be inserting 0's into RRDs.
Ran poller manually

./poller.php -h HOSTNAME -d

pertinent info: http://pastebin.com/QX9b8UAc

Contributor

tsumaru720 commented Feb 16, 2017

@laf Looks to still be inserting 0's into RRDs.
Ran poller manually

./poller.php -h HOSTNAME -d

pertinent info: http://pastebin.com/QX9b8UAc

@exarv

This comment has been minimized.

Show comment
Hide comment
@exarv

exarv Feb 16, 2017

Contributor

Maybe my PR #5930 fixes this?

Contributor

exarv commented Feb 16, 2017

Maybe my PR #5930 fixes this?

@tsumaru720

This comment has been minimized.

Show comment
Hide comment
@tsumaru720

tsumaru720 Feb 17, 2017

Contributor

Looking at comments on #5930 which suggests this is fixed so I updated from 872eac1 to 1140be1 and it does seem to be working again

Contributor

tsumaru720 commented Feb 17, 2017

Looking at comments on #5930 which suggests this is fixed so I updated from 872eac1 to 1140be1 and it does seem to be working again

@murrant murrant closed this in #5923 Feb 24, 2017

murrant added a commit that referenced this issue Feb 24, 2017

refactor: Updated storing of sensors data to be used in unix-agent #5904
 (#5923)

* refactor: Updated storing of sensors data to be used in unix-agent #5904

* updated from scrut output

* another scrut update

* added missing dbUpdate

* small change

* small change

@geordish geordish reopened this Mar 4, 2017

@geordish

This comment has been minimized.

Show comment
Hide comment
@geordish

geordish Mar 4, 2017

Contributor

18:25 < Demented-Idiot> how do i re-open an issue? #5904 came back :(

Please provide all information asked for here: http://docs.librenms.org/Support/FAQ/#faq20

Contributor

geordish commented Mar 4, 2017

18:25 < Demented-Idiot> how do i re-open an issue? #5904 came back :(

Please provide all information asked for here: http://docs.librenms.org/Support/FAQ/#faq20

laf added a commit to laf/librenms that referenced this issue Mar 4, 2017

@murrant murrant closed this in #6089 Mar 11, 2017

murrant added a commit that referenced this issue Mar 11, 2017

fix: Record actual sensor value for unix-agent hddtemp #5904 (#6089)
* fix: Record actual sensor value for unix-agent hddtemp #5904

* Update hddtemp.inc.php

* Update hddtemp.inc.php
@lock

This comment has been minimized.

Show comment
Hide comment
@lock

lock bot May 17, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed.

lock bot commented May 17, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed.

@librenms librenms locked as resolved and limited conversation to collaborators May 17, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.