Skip to content

Render CDB support field in sfputil CLI output#4705

Open
bgallagher-nexthop wants to merge 1 commit into
sonic-net:masterfrom
nexthop-ai:render-cdb-support-field-in-sfputil-cli
Open

Render CDB support field in sfputil CLI output#4705
bgallagher-nexthop wants to merge 1 commit into
sonic-net:masterfrom
nexthop-ai:render-cdb-support-field-in-sfputil-cli

Conversation

@bgallagher-nexthop

Copy link
Copy Markdown
Contributor

What I did

Add a line to the sfputil show eeprom output that indicates whether the transceiver advertises CDB support.

How I did it

sonic-net/sonic-platform-common#721 adds a field to get_transceiver_info that indicates whether the transceiver advertises CDB support.

This PR just renders that field in the CLI output.

How to verify it

Extended existing unit-test.

Previous command output (if the output of a command-line utility has changed)

~$ sudo sfputil show eeprom -p Ethernet144
Ethernet144: SFP EEPROM detected
        Active App Selection Host Lane 1: 1
        Active App Selection Host Lane 2: 1
        Active App Selection Host Lane 3: 1
        Active App Selection Host Lane 4: 1
        Active App Selection Host Lane 5: 1
        Active App Selection Host Lane 6: 1
        Active App Selection Host Lane 7: 1
        Active App Selection Host Lane 8: 1
        Application Advertisement: 400GAUI-4-L C2M (Annex 120G) - Host Assign (0x11) - 400GBASE-DR4 (Cl 124) - Media Assign (0x11)
                                   800GAUI-8 L C2M (Annex 120G) - Host Assign (0x1) - 800GBASE-DR8 (Clause 124) - Media Assign (0x1)
        CMIS Revision: 5.2
        Connector: MPO 1x12
        Encoding: N/A
        Extended Identifier: Power Class 5 (8.5W Max)
        Extended RateSelect Compliance: N/A
        Hardware Revision: 4.0
        Host Lane Count: 4
        Identifier: OSFP 8X Pluggable Transceiver
        Length Cable Assembly(m): 0.0
        Media Interface Technology: 1310 nm DFB
        Media Lane Count: 4
        Nominal Bit Rate(100Mbs): N/A
        Specification compliance: sm_media_interface
        Vendor Date Code(YYYY-MM-DD Lot): 2025-09-01
        Vendor Name: Amphenol
        Vendor OUI: 78-a7-14
        Vendor PN: OP13LD8-005D
        Vendor Rev: 10
        Vendor SN: 2508220085
        type_abbrv_name: OSFP-8X
        vdm_supported: False

New command output (if the output of a command-line utility has changed)

admin@gold220:~$ sudo sfputil show eeprom -p Ethernet144
Ethernet144: SFP EEPROM detected
        Active App Selection Host Lane 1: 1
        Active App Selection Host Lane 2: 1
        Active App Selection Host Lane 3: 1
        Active App Selection Host Lane 4: 1
        Active App Selection Host Lane 5: 1
        Active App Selection Host Lane 6: 1
        Active App Selection Host Lane 7: 1
        Active App Selection Host Lane 8: 1
        Application Advertisement: 400GAUI-4-L C2M (Annex 120G) - Host Assign (0x11) - 400GBASE-DR4 (Cl 124) - Media Assign (0x11)
                                   800GAUI-8 L C2M (Annex 120G) - Host Assign (0x1) - 800GBASE-DR8 (Clause 124) - Media Assign (0x1)
        CDB Supported: False
        CMIS Revision: 5.2
        Connector: MPO 1x12
        Encoding: N/A
        Extended Identifier: Power Class 5 (8.5W Max)
        Extended RateSelect Compliance: N/A
        Hardware Revision: 4.0
        Host Lane Count: 4
        Identifier: OSFP 8X Pluggable Transceiver
        Length Cable Assembly(m): 0.0
        Media Interface Technology: 1310 nm DFB
        Media Lane Count: 4
        Nominal Bit Rate(100Mbs): N/A
        Specification compliance: sm_media_interface
        Vendor Date Code(YYYY-MM-DD Lot): 2025-09-01
        Vendor Name: Amphenol
        Vendor OUI: 78-a7-14
        Vendor PN: OP13LD8-005D
        Vendor Rev: 10
        Vendor SN: 2508220085
        type_abbrv_name: OSFP-8X
        vdm_supported: False

Signed-off-by: Brian Gallagher <bgallagher@nexthop.ai>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@bgallagher-nexthop
bgallagher-nexthop marked this pull request as ready for review July 17, 2026 17:16
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@aditya-nexthop aditya-nexthop left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants