Details
Minor Description
Content is ready and tested against software and hardware devices.
Updated STEELHEAD-MIB and new ZXTM-MIB-SMIv2 will be added.
[user@box ~/SRC/observium-trunk-development]$ svn status
M includes/definitions/sensors.inc.php
M includes/discovery/os/linux.inc.php
A includes/discovery/sensors/steelhead-mib.inc.php
A includes/polling/os/steelhead.inc.php
M mibs/riverbed/STEELHEAD-MIB
A mibs/riverbed/ZXTM-MIB-SMIv2
[user@box ~/SRC/observium-trunk-development]$
Attachments
Activity
Thanks for the suggestions. Numeric OID's to pull model and version from are probably necessary in the polling/os/steelhead.inc.php file due to the EX models using a different MIB that I am yet to obtain a copy of.
re: copyright, the current license and in particular the attribution to Adam alone irks me, but I don't actually care one way or another.
I've attached another patch with fixes as suggested and standard Observium copyright.
Discovery debug output attached for both devices types in the same .txt file.
1. For sensor state definitions recomended use types with '-state' at end:
-$config['sensor_states']['steelhead-system-health']
|
+$config['sensor_states']['steelhead-system-state']
|
|
|
-$config['sensor_states']['steelhead-service-status']
|
+$config['sensor_states']['steelhead-service-state']
|
2. In snmp*() functions always (if possible) use explicit indication MIB and named oids:
-$value = snmp_get($device, $oid, "-Oqv", "", mib_dirs('riverbed'));
|
+$value = snmp_get($device, "optServiceStatus.0", "-Oqv", "STEELHEAD-MIB", mib_dirs('riverbed'));
|
3. Numeric oids only if named oids impossible!
4. For convert state names to values use state_string_to_numeric():
+$value = state_string_to_numeric('steelhead-service-state', $value);
|
5. Attach debug output for discovery devices (tho oses):
./discovery.php -d -h <riverbed_device>
|
./discovery.php -d -h <zeustm_device>
|
6. About @copyright, you should accept Observium License, in copyright will always be:
@copyright (C) 2006-2014 Adam Armstrong
|
If this is not possible, we will not accept the code in the main base.
Actually ignore the description, I had forgotten a couple of files and modifications.
Patch and proposed image attached.
Added in r5980.