sonic-net / sonic-platform-daemons

Platform module daemons for SONiC
Other
23 stars 152 forks source link

400ZR not linking up with latest SONiC master image #410

Closed mihirpat1 closed 10 months ago

mihirpat1 commented 10 months ago

Description

With the latest SONiC image, the link on 400ZR remains down.

Motivation and Context

The issue seems to have been introduced via https://github.com/sonic-net/sonic-platform-daemons/pull/400

The is_cmis_api API currently returns False for CCmis modules. Hence, the CMIS SM doesn't identify 400ZR and skips the port initialization as the desired application id is returned as None.

https://github.com/sonic-net/sonic-platform-daemons/blob/502c0b6622008363cb1ed6d1b7c85b4093997093/sonic-xcvrd/xcvrd/xcvrd.py#L134-L135

How Has This Been Tested?

Link up on 400ZR has been verified after switch boot-up.

root@sonic:/home/admin# show int status Ethernet144
  Interface                            Lanes    Speed    MTU    FEC         Alias    Vlan    Oper    Admin                                             Type    Asym PFC
-----------  -------------------------------  -------  -----  -----  ------------  ------  ------  -------  -----------------------------------------------  ----------
Ethernet144  145,146,147,148,149,150,151,152     400G   9100    N/A  Ethernet19/1  routed      up       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
root@sonic:/home/admin# show int status Ethernet160
  Interface                            Lanes    Speed    MTU    FEC         Alias    Vlan    Oper    Admin                                             Type    Asym PFC
-----------  -------------------------------  -------  -----  -----  ------------  ------  ------  -------  -----------------------------------------------  ----------
Ethernet160  161,162,163,164,165,166,167,168     400G   9100    N/A  Ethernet21/1  routed      up       up  QSFP-DD Double Density 8X Pluggable Transceiver         N/A
root@sonic:/home/admin# show lldp table
Capability codes: (R) Router, (B) Bridge, (O) Other
LocalPort    RemoteDevice                  RemotePortID    Capability    RemotePortDescr
-----------  ----------------------------  --------------  ------------  ------------------------
Ethernet144  sonic                         Ethernet21/1    BR            Ethernet160
Ethernet160  sonic                         Ethernet19/1    BR            Ethernet144
--------------------------------------------------
Total entries displayed:  2
root@sonic:/home/admin# 

Additional Information (Optional)

MSFT ADO - 26098607

mihirpat1 commented 10 months ago

@tshalvi - Can you please help in reviewing this?

mssonicbld commented 8 months ago

Cherry-pick PR to 202311: https://github.com/sonic-net/sonic-platform-daemons/pull/428