search

ap0calypse / bazinga

in-progress free blogging tool especially for coders
GNU General Public License v3.0
16 stars 0 forks source link

Argument "" isn't numeric #7

Open widhalmt opened 9 years ago

widhalmt commented 9 years ago

Hi!

I'm running your check against a TS-410U (Firmware: 4.0.2, Software: 4.0.2) and I'm getting the following error when I call the script with -a or -ct Option:

Argument "" isn't numeric in numeric ge (>=) at ./check_snmp_qnap_modified.pl line 534.

Can you give me a hint? It's working great on other systems.

Kind regards, Thomas

widhalmt commented 9 years ago

It's the same with Firmware: 4.1.0, Software: 4.1.0

ap0calypse commented 9 years ago

?!? what are you talking about? :D

widhalmt commented 9 years ago

Your qnap check script. :-) You linked here for an issue tracker.

ap0calypse commented 9 years ago

this project is a blog-tool....

widhalmt commented 9 years ago

I know, but the website I found for this qnap check perl script linked here. And as far as I remember there is already another issue for this script in this tracker. Otherwise I wouldn't have posted it here.

ap0calypse commented 9 years ago

can you give me the website adress linking here please? I really have NO idea what you are talking about. ;P

widhalmt commented 9 years ago

See http://enterranit.myqnapcloud.com/wordpress/nagios-uberwachung-eines-qnap-nas-uber-snmp/ at "bekannte Fehlermeldungen". Reading the article again it might be that the author just had the same error in his script like you and linked to you issue to show people how to resolve it. If you're not familiar with this script please accept my apologies and close this issue and the other one. I'm sorry.

ap0calypse commented 9 years ago

I'm not familiar with this script, sorry. but if you show me the part of the qnap-script that is affected, I could look over it and maybe help you :)

I find it interesting though, that this page links to this project .... strange.

ap0calypse commented 9 years ago

I just looked over the qnap script and I guess I found the little mistake.

Maybe change this in line 533: $cpu_temp =~ s/\s+.*//g;

to this: $cpu_temp =~ s/\s+.*//g; $cpu_temp = int($cpu_temp);

the later if-statement checks if the value is numeric, and doing that int()-stuff, maybe helps.

widhalmt commented 9 years ago

Thank you very much for your help! This is what I love in open source products (and nice people like you. :-) )

Unfortunately it didn't do the trick. I will try to contact the original author and might look into the script myself.

Thank you again, your trying to help me and being nice really made my day. Sometimes it just feels good when someone offers you help without sending you an invoice.

Have a nice day, Thomas

http://www.widhalm.or.at GnuPG : 6265BAE6 Skype: widhalmt Jabber: widhalmt@widhalm.or.at Threema: H7AV7D33 Red Hat Certified Engineer, CAcert "Experienced Assurer", GSIntroducer (GSWoT:AT57)

ap0calypse commented 9 years ago

pity ... well I guess there is something else then :P

maybe the vars don't get initialised under some circumstances.

have a nice day too :) greetings from tirol to salzburg :P