-
Notifications
You must be signed in to change notification settings - Fork 673
ADD 100G lambda MSA Types #1328
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
base: master
Are you sure you want to change the base?
Conversation
ADD 100G lambda MSA Types (100G-LR, 100G-FR, 4X100G-LR, 4X100G-FR)...
Do we want to continue to define new identity refs for all the possible breakouts or should we make this leaf truly the PMD and add a new leaf that is the number of PMD instances supported by the transceiver? |
Hi @proberts2022 , I agree that it would be much more efficient and closer to the standard to separate the Nx in the PMD types. I guess the main historical reason for keeping e.g. the NX in the type is that pluggable datasheets usually include that string in the specifications, so it is easy to find and search. The donwside it that is a much important change than adding a bunch of identity refs and the new field will take some time to get into the implementations. What to you think @dplore ? |
We have So I think we should not have per-breakout identities for the PMDs. Thoughts? |
@dplore
IMO, the components/component/port/breakout-group is focused on the C2M PMA covering the host interface between the router and the transceiver. In the case of grey optics, for every separate C2M PMA on the host interface, there is one PMD on the media interface. And in this case, the statement that the breakout-groups configuration can also apply to indicate the number of PMD may work. However, for coherent transceivers, there can be breakout configurations for the host interface that the transceiver then multiplexes into one PMD for the media interface. So maybe we leave coherent transceivers as a special case but it would impact (3) above and we would have to add a special note in the leaf description. |
@ejbrever @ahsaanyousaf can you comment? |
@oscargdd will share link(s) to example transceivers we want to represent in OC to help guide us on how they should be represented in OC. |
@dplore Please find a couple of examples of pluggables that need to be modeled: |
|
||
identity ETH_4X100GBASE_FR { | ||
base ETHERNET_PMD_TYPE; | ||
description "Ethernet compliance code: 4x100GBASE_FR"; |
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.
This PMD is same as 400GBASE_DR4?
|
||
identity ETH_100GBASE_FR { | ||
base ETHERNET_PMD_TYPE; | ||
description "Ethernet compliance code: 100GBASE_FR"; |
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.
This is same as line 1280?
|
||
identity ETH_4X100GBASE_LR { | ||
base ETHERNET_PMD_TYPE; | ||
description "Ethernet compliance code: 4x100GBASE_LR"; |
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.
So there is no standard for this one, should this be called 400GBASE_DR4++?
/gcbrun @ahsaanyousaf and @proberts2022 can you each provide your opinion of what the PMD assignment should be for these two transceivers? 2x100: https://apps.juniper.net/hct/model/?component=QDD-2X100G-LR4 4x100: https://edgeoptic.com/products/juniper/juniper-400g-200g-25g-compatible-transceivers/qdd-4x100g-fr/ |
No major YANG version changes in commit 58fdf1a |
2x100: https://apps.juniper.net/hct/model/?component=QDD-2X100G-LR4 4x100: https://edgeoptic.com/products/juniper/juniper-400g-200g-25g-compatible-transceivers/qdd-4x100g-fr/ |
I thought Peter's proposal was to avoid things like "2x" in the PMD identities. So wouldn't that first one be this? |
Change Scope
The requested change is to include new PMD types:
100G lambda MSA Types: 100G-LR, 100G-FR, 4X100G-LR, 4X100G-FR
See https://100glambda.com/
2X100G-LR4 is also added.
Pluggables with those PMDs have been found in the live network and were not being properly reported today as the PMD code was missing.
Platform Implementations