Skip to content

Commit

Permalink
OcNOS transceiver temperature display fix (librenms#16460)
Browse files Browse the repository at this point in the history
* OcNOS transceiver temperature display fix
Temperatures were not displayed in the transceiver area.  Update the group to fix that issue.

* Update test data
  • Loading branch information
murrant authored Sep 30, 2024
1 parent 615a5ab commit dbfb0a6
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 12 deletions.
2 changes: 1 addition & 1 deletion includes/discovery/sensors/temperature/ocnos.inc.php
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
'entPhysicalIndex' => $cmmStackUnitIndex * 10000 + $cmmTransIndex,
'entPhysicalIndex_measured' => 'port',
'user_func' => null,
'group' => $ifName,
'group' => 'transceiver',
]));

continue 2; // common across channels
Expand Down
22 changes: 11 additions & 11 deletions tests/data/ocnos_s9600-32x.json
Original file line number Diff line number Diff line change
Expand Up @@ -4039,7 +4039,7 @@
"sensor_index": "1.10",
"sensor_type": "transceiver",
"sensor_descr": "ce5 xcvr temperature",
"group": "ce5",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 35.96,
Expand All @@ -4064,7 +4064,7 @@
"sensor_index": "1.11",
"sensor_type": "transceiver",
"sensor_descr": "ce6 xcvr temperature",
"group": "ce6",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 38.63,
Expand All @@ -4089,7 +4089,7 @@
"sensor_index": "1.12",
"sensor_type": "transceiver",
"sensor_descr": "ce7 xcvr temperature",
"group": "ce7",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 32.89,
Expand All @@ -4114,7 +4114,7 @@
"sensor_index": "1.13",
"sensor_type": "transceiver",
"sensor_descr": "ce8 xcvr temperature",
"group": "ce8",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 30.82,
Expand All @@ -4139,7 +4139,7 @@
"sensor_index": "1.14",
"sensor_type": "transceiver",
"sensor_descr": "ce9 xcvr temperature",
"group": "ce9",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 29.49,
Expand All @@ -4164,7 +4164,7 @@
"sensor_index": "1.21",
"sensor_type": "transceiver",
"sensor_descr": "ce16 xcvr temperature",
"group": "ce16",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 31.23,
Expand All @@ -4189,7 +4189,7 @@
"sensor_index": "1.22",
"sensor_type": "transceiver",
"sensor_descr": "ce17 xcvr temperature",
"group": "ce17",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 30.38,
Expand All @@ -4214,7 +4214,7 @@
"sensor_index": "1.23",
"sensor_type": "transceiver",
"sensor_descr": "ce18 xcvr temperature",
"group": "ce18",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 30.23,
Expand All @@ -4239,7 +4239,7 @@
"sensor_index": "1.24",
"sensor_type": "transceiver",
"sensor_descr": "ce19 xcvr temperature",
"group": "ce19",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 30.15,
Expand All @@ -4264,7 +4264,7 @@
"sensor_index": "1.3",
"sensor_type": "transceiver",
"sensor_descr": "xe1 xcvr temperature",
"group": "xe1",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 27.34,
Expand All @@ -4289,7 +4289,7 @@
"sensor_index": "1.9",
"sensor_type": "transceiver",
"sensor_descr": "ce4 xcvr temperature",
"group": "ce4",
"group": "transceiver",
"sensor_divisor": 100,
"sensor_multiplier": 1,
"sensor_current": 36.15,
Expand Down

0 comments on commit dbfb0a6

Please sign in to comment.