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
Documenting mandatory profile for OI applications #130
Comments
Here the JSON example about OI application sent by Esther. Thanks Esther! |
From Esther:
|
In case of parallel multi-band amplifiers one can describe the individual amplifiers as proposed, but there is an additional splitter/combiner element that needs to be modeled. Suggest to re-phrase:
amplifier OMS-element |
@ggrammel your commment is a good point: it looks as if we missed the splitters and combiners. However I'd prefer to model them inside the OMS-element as they are actually an essential part of a multi-band amplifier. Thus we can
what others think? |
If I recalled well, we have discussed this point and agreed for option 2 Could you please check whether this is properly documented in the I-D text and/or YANG descriptions? |
@italobusi @egriseri: I've also have some reminds about this discussion and decision for option 2, that was also the chosen option also in other cases of our model, e.g. for protection exploiting OI in the connectivity matrix. But we have not documented in the I-D this decision . Take into account DGE section is still to be completed with text in issue #153 and so we can add also this decision in the context of update for IETF119. Is it enough the text provided by Enrico or we should elaborate more ? |
The YANG model for OI has been updated to allow profiling the YANG model for other applications besides the original OI applications
However, in order to foster interoperability, there is a need to document the mandatory profile for OI applications: the profile should indicate the attributes which are optional in the model because not needed for non-OI applications but are required to support the OI applications
The text was updated successfully, but these errors were encountered: