mirror of
https://github.com/librenms/librenms.git
synced 2024-10-07 16:52:45 +00:00
Fixed Code in SRX Session Graphing (#8816)
DO NOT DELETE THIS TEXT #### Please note > Please read this information carefully. You can run `./scripts/pre-commit.php` to check your code before submitting. - [x ] Have you followed our [code guidelines?](http://docs.librenms.org/Developing/Code-Guidelines/) #### Testers If you would like to test this pull request then please run: `./scripts/github-apply <pr_id>`, i.e `./scripts/github-apply 5926` Fixed an issue in the SRX session graphing code which caused Junos BGP sessions to alert as down even though they were established. Previous pull to add the session graphs: #8815
This commit is contained in:
committed by
Neil Lathwood
parent
05c03a2290
commit
d0efc742f3
@@ -3,20 +3,21 @@
|
||||
use LibreNMS\RRD\RrdDefinition;
|
||||
|
||||
$oid_list = 'jnxJsSPUMonitoringCurrentFlowSession.0';
|
||||
$junos = snmp_get_multi($device, $oid_list, '-OUQs', 'JUNIPER-SRX5000-SPU-MONITORING-MIB');
|
||||
$srx_sess_data = snmp_get_multi($device, $oid_list, '-OUQs', 'JUNIPER-SRX5000-SPU-MONITORING-MIB');
|
||||
|
||||
if (is_numeric($junos[0]['jnxJsSPUMonitoringCurrentFlowSession'])) {
|
||||
if (is_numeric($srx_sess_data[0]['jnxJsSPUMonitoringCurrentFlowSession'])) {
|
||||
$tags = array(
|
||||
'rrd_def' => RrdDefinition::make()->addDataset('spu_flow_sessions', 'GAUGE', 0),
|
||||
);
|
||||
$fields = array(
|
||||
'spu_flow_sessions' => $junos[0]['jnxJsSPUMonitoringCurrentFlowSession'],
|
||||
'spu_flow_sessions' => $srx_sess_data[0]['jnxJsSPUMonitoringCurrentFlowSession'],
|
||||
);
|
||||
|
||||
data_update($device, 'junos_jsrx_spu_sessions', $tags, $fields);
|
||||
|
||||
$graphs['junos_jsrx_spu_sessions'] = true;
|
||||
echo ' Flow Sessions';
|
||||
unset($srx_sess_data);
|
||||
}
|
||||
|
||||
$version = snmp_get($device, 'jnxVirtualChassisMemberSWVersion.0', '-Oqv', 'JUNIPER-VIRTUALCHASSIS-MIB');
|
||||
|
||||
Reference in New Issue
Block a user