mirror of
https://github.com/librenms/librenms.git
synced 2024-10-07 16:52:45 +00:00
fix: Standardise UPS divisors (#6942)
Set default divisors for all devices and remove per-os overrides for them. We should only use per-os overrides for non-standard behavior. High possibility for breakage here if devices not listed here have non-standard behavior.
This commit is contained in:
committed by
Neil Lathwood
parent
dcca74dcc4
commit
c874fc5b1e
@ -959,49 +959,28 @@ function get_device_divisor($device, $os_version, $sensor_type, $oid)
|
||||
} else {
|
||||
return 1;
|
||||
}
|
||||
} elseif ($sensor_type == 'voltage') {
|
||||
return 1;
|
||||
}
|
||||
} elseif (($device['os'] == 'huaweiups') && ($sensor_type == 'frequency')) {
|
||||
return 100;
|
||||
} elseif (($device['os'] == 'netmanplus') && ($sensor_type == 'load')) {
|
||||
return 1;
|
||||
} elseif (($device['os'] == 'netmanplus') && ($sensor_type == 'voltage')) {
|
||||
if (preg_match('/.1.3.6.1.2.1.33.1.2.5./', $oid)) {
|
||||
return 10;
|
||||
}
|
||||
return 1;
|
||||
} elseif ($device['os'] == 'generex-ups') {
|
||||
if ($sensor_type == 'load') {
|
||||
return 1;
|
||||
} elseif ($sensor_type == 'voltage' && !starts_with($oid, '.1.3.6.1.2.1.33.1.2.5.')) {
|
||||
return 1;
|
||||
}
|
||||
} elseif ($device['os'] == 'apc-mgeups') {
|
||||
if ($sensor_type == 'load') {
|
||||
return 1;
|
||||
} elseif ($sensor_type == 'voltage' && !starts_with($oid, '.1.3.6.1.2.1.33.1.2.5.')) {
|
||||
return 1;
|
||||
}
|
||||
} elseif ($device['os'] == 'ge-ups') {
|
||||
if ($sensor_type == 'load') {
|
||||
return 1;
|
||||
} elseif ($sensor_type == 'voltage' && !starts_with($oid, '.1.3.6.1.2.1.33.1.2.5.')) {
|
||||
return 1;
|
||||
}
|
||||
} elseif ($device['os'] == 'eaton-mgeups') {
|
||||
if ($sensor_type == 'load') {
|
||||
return 1;
|
||||
} elseif ($device['os'] == 'huaweiups') {
|
||||
if ($sensor_type == 'frequency') {
|
||||
return 100;
|
||||
}
|
||||
} elseif ($device['os'] == 'hpe-rtups') {
|
||||
if ($sensor_type == 'load') {
|
||||
return 1;
|
||||
} elseif ($sensor_type == 'voltage' && !starts_with($oid, '.1.3.6.1.2.1.33.1.2.5.') && !starts_with($oid, '.1.3.6.1.2.1.33.1.3.3.1.3')) {
|
||||
if ($sensor_type == 'voltage' && !starts_with($oid, '.1.3.6.1.2.1.33.1.2.5.') && !starts_with($oid, '.1.3.6.1.2.1.33.1.3.3.1.3')) {
|
||||
return 1;
|
||||
}
|
||||
}
|
||||
|
||||
return 10; //default
|
||||
// UPS-MIB Defaults
|
||||
|
||||
if ($sensor_type == 'load') {
|
||||
return 1;
|
||||
}
|
||||
|
||||
if ($sensor_type == 'voltage' && !starts_with($oid, '.1.3.6.1.2.1.33.1.2.5.')) {
|
||||
return 1;
|
||||
}
|
||||
|
||||
return 10;
|
||||
}
|
||||
|
||||
/**
|
||||
|
Reference in New Issue
Block a user