Skip to content

CPO common cpoutil commands - #2498

Draft
KroosMicas wants to merge 1 commit into
sonic-net:masterfrom
KroosMicas:master_4
Draft

CPO common cpoutil commands#2498
KroosMicas wants to merge 1 commit into
sonic-net:masterfrom
KroosMicas:master_4

Conversation

@KroosMicas

Copy link
Copy Markdown

add CPO common cpoutil commands

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

Signed-off-by: duanchunming <kroos@micasnetworks.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@KroosMicas
KroosMicas marked this pull request as draft August 12, 2026 09:20
@zili11720

Copy link
Copy Markdown

Hi @KroosMicas,

I have 2 questions regarding the cpoutil show commands:

  1. I don’t see how vendor-specific fields can be integrated into the output.
    In sfputil, DOM formatting is driven by known maps, so unmapped fields are not printed. If cpoutil follows that pattern, vendors will need a defined way to contribute labels/units for platform-specific fields.

I suggest supporting vendor-specific format maps that are merged into cpoutil show output, with the maps owned by the platform rather than hardcoded in the common CLI. This reuses the idea from #2317 adapted for cpoutil.
Can the HLD clarify how vendor specific fields should be integrated?

  1. Today for pluggables we have two types of show commands:
  • sfputil — direct hardware access (sonic-utilities/sfputil/main.py)
  • show interfaces transceiver … - reads STATE_DB only (sonic-utilities/scripts/sfpshow)
    Examples:
    show interfaces transceiver eeprom EthernetX -d
    show interfaces transceiver status EthernetX
    This HLD covers a CPO analogue of sfputil (cpoutil). I don’t see a STATE_DB-backed 'show interfaces …' path for CPO in scope.

Do we plan to extend / add show interfaces … commands for CPO that rely on STATE_DB only (like sfpshow)?
If yes, the first question applies here as well for DOM/STATUS maps.

Thanks


#### 8.2.6.cpoutil config els reset ELS_INDEX

Function: Trigger ELS hardware reset signal.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

what about the reset of some specific laser on a certain ELS ? Some vendors can support it


**Out of scope:**

1. All `cpoutil firmware` subcommands (see separate linked design document:[github.com/sonic-net/SONiC/pull/2484](https://github.com/sonic-net/SONiC/pull/2484))

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

please refer to the Nvidia HLD on cpoutil for FW management - #2489

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.

4 participants