[Nut-upsdev] MGE MIB related bug

[Arnaud Quette]

Hi Gabor,

2011/11/22 Kiss Gabor (Bitman) <kissg at ssg.ki.iif.hu>:
> Dear Arnaud,
>
>> > That said, I've added and committed the above OID with the standard lowbatt
>> > mapping (1 = LB, 2 = <not LB>)
>> > Can you please test the trunk, r3318?
>
> Do you mean this?
> http://trac.networkupstools.org/projects/nut/changeset/3318/trunk/drivers/mge-mib.c

this is only the diff, but somehow also what I meant:
testing either this change in your tree, or the trunk with a revision
number that is at least 3318 (to be sure the modification is included)

>> Thanks.
>> I'll do in a week or so.
>> Unfortunately at this moment I'm quite busy.
>
> I added your new line to version 2.4.1 mgemib.h.
> (I have this source prepared on my disk, that was the laziest way
> to test your patch. :)
>
> The result is a bit strange:
> ups.status: OL OL

indeed

> More detailed log follows. (Relevant raw SNMP vars and upsc output)
>
> MG-SNMP-UPS-MIB::upsmgConfigLowBatteryTime.0 = INTEGER: 180
> MG-SNMP-UPS-MIB::upsmgBatteryRemainingTime.0 = INTEGER: 2791
> MG-SNMP-UPS-MIB::upsmgBatteryLowBattery.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgBatteryLowCondition.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgOutputOnBattery.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgOutputOnByPass.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgOutputUtilityOff.0 = INTEGER: no(2)
> battery.runtime: 2791.00
> ups.status: OL OL
>
> MG-SNMP-UPS-MIB::upsmgConfigLowBatteryTime.0 = INTEGER: 180
> MG-SNMP-UPS-MIB::upsmgBatteryRemainingTime.0 = INTEGER: 7430
> MG-SNMP-UPS-MIB::upsmgBatteryLowBattery.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgBatteryLowCondition.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgOutputOnBattery.0 = INTEGER: yes(1)
> MG-SNMP-UPS-MIB::upsmgOutputOnByPass.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgOutputUtilityOff.0 = INTEGER: yes(1)
> battery.runtime: 7430.00
> ups.status: OB OB
>
> MG-SNMP-UPS-MIB::upsmgConfigLowBatteryTime.0 = INTEGER: 9999
> MG-SNMP-UPS-MIB::upsmgBatteryRemainingTime.0 = INTEGER: 7430
> MG-SNMP-UPS-MIB::upsmgBatteryLowBattery.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgBatteryLowCondition.0 = INTEGER: yes(1)
> MG-SNMP-UPS-MIB::upsmgOutputOnBattery.0 = INTEGER: yes(1)
> MG-SNMP-UPS-MIB::upsmgOutputOnByPass.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgOutputUtilityOff.0 = INTEGER: yes(1)
> battery.runtime: 7430.00
> ups.status: OB LB OB
>
> MG-SNMP-UPS-MIB::upsmgConfigLowBatteryTime.0 = INTEGER: 9999
> MG-SNMP-UPS-MIB::upsmgBatteryRemainingTime.0 = INTEGER: 2791
> MG-SNMP-UPS-MIB::upsmgBatteryLowBattery.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgBatteryLowCondition.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgOutputOnBattery.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgOutputOnByPass.0 = INTEGER: no(2)
> MG-SNMP-UPS-MIB::upsmgOutputUtilityOff.0 = INTEGER: no(2)
> battery.runtime: 2791.00
> ups.status: OL OL
>
> Is this what you expected?

not at all.

> Or should I compile a version 2.6.2 driver?

yes, test the trunk to be sure of the code your testing.
I can't commit on the fix application you made, nor have much time to
spare on this...

> (The test UPS is available in this week only. Please reply soon.)

I'm doing my best, but just remember that I'm there supporting a user
from our (Eaton) major competitor ;-)

cheers,
Arnaud
-- 
Linux / Unix Expert R&D - Eaton - http://powerquality.eaton.com
Network UPS Tools (NUT) Project Leader - http://www.networkupstools.org/
Debian Developer - http://www.debian.org
Free Software Developer - http://arnaud.quette.free.fr/