OpenConfig Platform Transceiver HLD#1858
OpenConfig Platform Transceiver HLD#1858nakano-omw wants to merge 2 commits intosonic-net:masterfrom
Conversation
nakano-omw
force-pushed
the
openconfig-platform
branch
from
120c0be to
05ae53d
Compare
nakano-omw
force-pushed
the
openconfig-platform
branch
from
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.
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.
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
|
@zhangyanzhao I would like to see it added to SONiC 202505 Release |
Signed-off-by: Kanji Nakano <[email protected]>
|
/azp run |
|
No pipelines are associated with this pull request. |
|
@jeff-yin I have updated the HLD with the changes related to additional support for CRITICAL/WARNING threshold parameters. In addition, I have also submitted a code PR for the support. I would appreciate it if you could review the code. Thank you! |
|
@nakano-omw I added this feature to 202505 release. Can you please add the code PRs by referring to #806 ? Thanks. |
7 participants
OpenConfig support for Platform Transceiver.
Add support for following features: