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

Qnap missing temprature and fan speed #5910

Closed
anden4380 opened this Issue Feb 15, 2017 · 33 comments

Comments

Projects
None yet
2 participants
@anden4380

anden4380 commented Feb 15, 2017

DO NOT DELETE THIS INFORMATION.

Please read this information carefully.

GitHub issues is for bugs, please do not post issues asking for help or how to do X, Y or Z.
You can use our irc channel ##librenms on freenode to ask questions or our community site.
If you have a feature request please post this on our community site.

Please confirm each of the sections below by putting an x in the box like [x].

  • Is your install up to date? Updating your install
    Please do not submit an issue if your install is not up to date within the last 24 hours or on a stable monthly release.
  • Please include all of the information between the ==================================== section of ./validate.php which you can run from the cli.
  • Please provide ALL info asked for here.
  • [X 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.

validate.php

==========================================================

[OK] Database connection successful

image

I have a Qnap NAS and can see some information are missing

I have given you snmp access

Device: 89.239.195.168
Version: 2c
Community string: librenms
Port: 1161

Thanks

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

Thanks for access. I've used your device and can confirm that all works fine on the latest master. As you've not provided the required output from validate we can't confirm the version you run. I suggest however if you haven't that you update.

image

Member

laf commented Feb 16, 2017

Thanks for access. I've used your device and can confirm that all works fine on the latest master. As you've not provided the required output from validate we can't confirm the version you run. I suggest however if you haven't that you update.

image

@laf laf closed this Feb 16, 2017

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

can not get it to work and it a new installation with the news master

==========================================================
Component | Version
--------- | -------
LibreNMS | d06aaaa
DB Schema | 164
PHP | 7.0.13-0ubuntu0.16.04.1
MySQL | 10.0.29-MariaDB-0ubuntu0.16.04.1
RRDTool | 1.5.5
SNMP | NET-SNMP 5.7.3

[OK] Database connection successful

anden4380 commented Feb 16, 2017

can not get it to work and it a new installation with the news master

==========================================================
Component | Version
--------- | -------
LibreNMS | d06aaaa
DB Schema | 164
PHP | 7.0.13-0ubuntu0.16.04.1
MySQL | 10.0.29-MariaDB-0ubuntu0.16.04.1
RRDTool | 1.5.5
SNMP | NET-SNMP 5.7.3

[OK] Database connection successful

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

I have other installation at my work and also have qnap there with same problem and are also up to date

there must be something missing

anden4380 commented Feb 16, 2017

I have other installation at my work and also have qnap there with same problem and are also up to date

there must be something missing

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

You need to provide the poller and discovery info we ask for in point 3 above.

Member

laf commented Feb 16, 2017

You need to provide the poller and discovery info we ask for in point 3 above.

@anden4380

This comment has been minimized.

Show comment
Hide comment

anden4380 commented Feb 16, 2017

@laf laf reopened this Feb 16, 2017

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

That's not been run with the -d commands as per point 3

Member

laf commented Feb 16, 2017

That's not been run with the -d commands as per point 3

@anden4380

This comment has been minimized.

Show comment
Hide comment

anden4380 commented Feb 16, 2017

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member
.*.4.1.246*.*.* = No Such Instance currently exists at this OID
.*.4.1.246*.*.* = No Such Instance currently exists at this OID
.*.4.1.246*.*.* = No Such Instance currently exists at this OID
.*.4.1.246*.*.* = No Such Instance currently exists at this OID

That explains it.

I get this from your device:

.*.4.1.246*.0 = 42
.*.4.1.246*.1.3.1 = "42 C/107 F"
.*.4.1.246*.1.3.2 = "42 C/107 F"
.*.4.1.246*.1.3.3 = "42 C/107 F"
.*.4.1.246*.1.3.4 = "43 C/109 F"

Maybe it's worth you deleting the device and re-adding just to see. Also make sure you don't have anything in /etc/snmp/snmp.conf (not snmpd.conf)

Member

laf commented Feb 16, 2017

.*.4.1.246*.*.* = No Such Instance currently exists at this OID
.*.4.1.246*.*.* = No Such Instance currently exists at this OID
.*.4.1.246*.*.* = No Such Instance currently exists at this OID
.*.4.1.246*.*.* = No Such Instance currently exists at this OID

That explains it.

I get this from your device:

.*.4.1.246*.0 = 42
.*.4.1.246*.1.3.1 = "42 C/107 F"
.*.4.1.246*.1.3.2 = "42 C/107 F"
.*.4.1.246*.1.3.3 = "42 C/107 F"
.*.4.1.246*.1.3.4 = "43 C/109 F"

Maybe it's worth you deleting the device and re-adding just to see. Also make sure you don't have anything in /etc/snmp/snmp.conf (not snmpd.conf)

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

anden4380 commented Feb 16, 2017

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

I don't know what else to suggest. If this is the same device you've given me access to then it works as expected. I did put a comment about snmp.conf but you've not responded to that.

Member

laf commented Feb 16, 2017

I don't know what else to suggest. If this is the same device you've given me access to then it works as expected. I did put a comment about snmp.conf but you've not responded to that.

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

anden4380 commented Feb 16, 2017

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

On the librenms server.

I seem to have just lost access to your device, I'm making some code changes to take into account a potential issue, any chance you can re-enable it?

Member

laf commented Feb 16, 2017

On the librenms server.

I seem to have just lost access to your device, I'm making some code changes to take into account a potential issue, any chance you can re-enable it?

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

thanks :-)

it works now :-) some how i think my NAS i overload from outside on SNMP :-)

anden4380 commented Feb 16, 2017

thanks :-)

it works now :-) some how i think my NAS i overload from outside on SNMP :-)

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

Sorry :(

Can you try this patch please:

https://www.irccloud.com/pastebin/raw/FaAUgFAa

Member

laf commented Feb 16, 2017

Sorry :(

Can you try this patch please:

https://www.irccloud.com/pastebin/raw/FaAUgFAa

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

is it merge?

anden4380 commented Feb 16, 2017

is it merge?

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

anden4380 commented Feb 16, 2017

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

No you need to run that patch I linked above.

Member

laf commented Feb 16, 2017

No you need to run that patch I linked above.

laf added a commit to laf/librenms that referenced this issue Feb 16, 2017

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

have run daily and discover and still same :-(

anden4380 commented Feb 16, 2017

have run daily and discover and still same :-(

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 commented Feb 16, 2017

image

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

have you applied the patch?

Member

laf commented Feb 16, 2017

have you applied the patch?

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

anden4380 commented Feb 16, 2017

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

!!! From above!

Can you try this patch please:

https://www.irccloud.com/pastebin/raw/FaAUgFAa

Member

laf commented Feb 16, 2017

!!! From above!

Can you try this patch please:

https://www.irccloud.com/pastebin/raw/FaAUgFAa

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

anden4380 commented Feb 16, 2017

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member
cd /opt/librenms/
curl https://www.irccloud.com/pastebin/raw/FaAUgFAa -o patch
git apply patch

Then run a discovery for that device.

Member

laf commented Feb 16, 2017

cd /opt/librenms/
curl https://www.irccloud.com/pastebin/raw/FaAUgFAa -o patch
git apply patch

Then run a discovery for that device.

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

mutch better :-)

image

anden4380 commented Feb 16, 2017

mutch better :-)

image

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

Please wait for 15 mins and see.

Member

laf commented Feb 16, 2017

Please wait for 15 mins and see.

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

i am fighting with my qnap i wil lgive feed back then i have info

anden4380 commented Feb 16, 2017

i am fighting with my qnap i wil lgive feed back then i have info

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

working :-)

image

anden4380 commented Feb 16, 2017

working :-)

image

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

I'll pr this up, you'll need to remove that change by doing git checkout includes/snmp.inc.php includes/discovery/sensors/temperatures/qnap.inc.php - the temps will break but once the fix is merged in then you will start getting data again.

Member

laf commented Feb 16, 2017

I'll pr this up, you'll need to remove that change by doing git checkout includes/snmp.inc.php includes/discovery/sensors/temperatures/qnap.inc.php - the temps will break but once the fix is merged in then you will start getting data again.

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

anden4380 commented Feb 16, 2017

@laf

This comment has been minimized.

Show comment
Hide comment
@laf

laf Feb 16, 2017

Member

That patch didn't contain the fix but the pull request does.

Member

laf commented Feb 16, 2017

That patch didn't contain the fix but the pull request does.

@anden4380

This comment has been minimized.

Show comment
Hide comment
@anden4380

anden4380 Feb 16, 2017

anden4380 commented Feb 16, 2017

@laf laf closed this in #5925 Feb 17, 2017

laf added a commit that referenced this issue Feb 17, 2017

fix: Updated qnap sensor code to be more generic #5910 (#5925)
* fix: Updated qnap sensor code to be more generic #5910

* small change to check for valid data

* small code change
@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.

@lock lock bot 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.