-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
OpenConfig Platform Transceiver HLD #1858
base: master
Are you sure you want to change the base?
Conversation
120c0be
to
05ae53d
Compare
05ae53d
to
18e936e
Compare
This HLD was reviewed by the UMF WG on November 27, 2024. |
- output-power | ||
- input-power | ||
- laser-bias-current | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about high/low warning/critical thresholds for temperature and voltage? These should also be encoded in transceiver DOM values.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your suggestion.
We found that we can fetch the value of thresholds.
As a first step, we are thinking to implement the functions to get the value of the following thresholds.
components/component[transceiver_XXX]/transceiver/thresholds/threshold[CRITICAL]/state/output-power-upper components/component[transceiver_XXX]/transceiver/thresholds/threshold[CRITICAL]/state/output-power-lower components/component[transceiver_XXX]/transceiver/thresholds/threshold[CRITICAL]/state/input-power-upper components/component[transceiver_XXX]/transceiver/thresholds/threshold[CRITICAL]/state/input-power-lower components/component[transceiver_XXX]/transceiver/thresholds/threshold[CRITICAL]/state/laser-bias-current-upper components/component[transceiver_XXX]/transceiver/thresholds/threshold[CRITICAL]/state/laser-bias-current-lower
components/component[transceiver_XXX]/transceiver/thresholds/threshold[WARNING]/state/output-power-upper components/component[transceiver_XXX]/transceiver/thresholds/threshold[WARNING]/state/output-power-lower components/component[transceiver_XXX]/transceiver/thresholds/threshold[WARNING]/state/input-power-upper components/component[transceiver_XXX]/transceiver/thresholds/threshold[WARNING]/state/input-power-lower components/component[transceiver_XXX]/transceiver/thresholds/threshold[WARNING]/state/laser-bias-current-upper components/component[transceiver_XXX]/transceiver/thresholds/threshold[WARNING]/state/laser-bias-current-lower
OpenConfig support for Platform Transceiver.
Add support for following features: