openBackhaul / wireInterface

Technology specific interface definition for Ethernet PHY interfaces according to IEEE 802.3
Apache License 2.0
2 stars 1 forks source link

supportedAlarmList to get DefaultValue and * #33

Closed openBackhaul closed 2 years ago

openBackhaul commented 3 years ago

The current concept of supportedAlarmList:

This concept doesn't work, because it is not assured that devices support the mandatory alarms. As a consequence they could not be listed in the Capability attribute. In worst case (happened at least for HybridMwStructure and PureEthernetStructure) the device does not support any alarm at all at this interface.

Because it is a list attribute, but does not have a default value, the attribute doesn't show up in the implementation in this case.

Proposal: All supportedAlarmList attributes shall have multiplicity and default value "Supported Alarms Not Yet Defined." supportedAlarmList : String [] = "Supported Alarms Not Yet Defined."

openBackhaul commented 3 years ago

Decision made on the 5G-xhaul call on 28th of July 2021: The supportedAlarmList attribute shall have multiplicity and default value "Supported alarms not yet defined." (supportedAlarmList : String [] = "Supported alarms not yet defined.") UML and YANG have to be corrected.

Further on, the comment shall be corrected from "Available alarms to be listed. Mandatory: 'rxLos'. Optional: 'txFault', 'tempHigh', 'tempLow', 'rxLevelHigh', 'rxLevelLow', 'vccHigh', 'vccLow', 'txBiasHigh', 'txBiasLow', 'txPowerHigh', 'txPowerLow', 'laserTempHigh', 'laserTempLow', 'tecCurrentHigh', 'tecCurrentLow'. Further alarms might be added by the device." towards "Available alarms to be listed by the vendor.".

openBackhaul commented 2 years ago

Got obsolete due to issue #39.