Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

supportedAlarmList to get DefaultValue and * #42

Closed
openBackhaul opened this issue Jul 27, 2020 · 2 comments
Closed

supportedAlarmList to get DefaultValue and * #42

openBackhaul opened this issue Jul 27, 2020 · 2 comments
Assignees
Labels
duplicate This issue or pull request already exists

Comments

@openBackhaul
Copy link
Owner

The current concept of supportedAlarmList:

  • Defining a couple of alarms, which are mandatory to be supported in the comment,
  • defining the minimum multiplicity according to the number of mandatory alarms
  • (because not possible to have default, when minimum multiplicity > 0) having no default value.

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 openBackhaul self-assigned this Jul 27, 2020
@openBackhaul openBackhaul removed the bug label Nov 18, 2020
@openBackhaul
Copy link
Owner Author

openBackhaul commented Jun 1, 2021

Decision on the 5G-xhaul call on 16th of June
The EthernetContainerCapability::supportedAlarmList attribute shall be changed from [2..] to [] multiplicity, the default value "Supported Alarms Not Yet Defined." shall be added and the formulation "Mandatory:'framingIsFaulty' and 'containerIsDown'. " shall be removed from the comment.

supportedAlarmList : String [*] = "Supported Alarms Not Yet Defined."

@openBackhaul
Copy link
Owner Author

obsoleted by issue #47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

1 participant