Skip to content

HLD: Transceiver Firmware Management Enhancement#2304

Open
rohit-nexthop wants to merge 3 commits into
sonic-net:masterfrom
nexthop-ai:rohit.XcvrFwMgmt
Open

HLD: Transceiver Firmware Management Enhancement#2304
rohit-nexthop wants to merge 3 commits into
sonic-net:masterfrom
nexthop-ai:rohit.XcvrFwMgmt

Conversation

@rohit-nexthop
Copy link
Copy Markdown

Added HLD for enhancement to the sfputil command-line utility to support firmware management operations for optical transceivers in SONiC.

Signed-off-by: Rohit Sharma <rohit@nexthop.ai>
@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla Bot commented Apr 22, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

No pipelines are associated with this pull request.

@rohit-nexthop
Copy link
Copy Markdown
Author

@prgeor

@eddyk-nvidia
Copy link
Copy Markdown

What happens if interfaces mentioned in the list a mapped to the same transceiver ? Today we can have up to 8 interfaces/ports mapped to the same module. Tomorrow - even more. The problem exists even today. When using a single interface name we can update up to 8 times the same module. Moreover, impacting not relevant interfaces mapped to the same module. Do we want to put some limitation, e.g. allow the upgrade only to the port with lower interface number. Or at least provide some warning information about a blust radius of some command (up to 8 interfaces can be impacted today)

@eddyk-nvidia
Copy link
Copy Markdown

@rohit-nexthop - I appologize - it looks like I erroneously closed this PR after putting a comment. Don't know how it happened.

@eddyk-nvidia eddyk-nvidia reopened this Apr 27, 2026
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

No pipelines are associated with this pull request.

@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

No pipelines are associated with this pull request.

@rohit-nexthop
Copy link
Copy Markdown
Author

What happens if interfaces mentioned in the list a mapped to the same transceiver ? Today we can have up to 8 interfaces/ports mapped to the same module. Tomorrow - even more. The problem exists even today. When using a single interface name we can update up to 8 times the same module. Moreover, impacting not relevant interfaces mapped to the same module. Do we want to put some limitation, e.g. allow the upgrade only to the port with lower interface number. Or at least provide some warning information about a blust radius of some command (up to 8 interfaces can be impacted today)

Thanks for reviewing the HLD. Thats a fair observation. We are handling it using a Transceiver Deduplication mechanism based on transceiver serial number. I've updated the HLD with the Transceiver Deduplication to reflect this.

Signed-off-by: Rohit Sharma <rohit@nexthop.ai>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

No pipelines are associated with this pull request.

Comment thread doc/sfputil/fw-mgmt-enhancement.md
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new SONiC HLD for enhancing sfputil with transceiver firmware management workflows, including filtered firmware version display and multi-port firmware download/upgrade support.

Changes:

  • Defines new sfputil show fwversion, firmware download, and firmware upgrade options for interface and vendor PN filtering.
  • Documents architecture, platform API usage, parallel execution, progress reporting, limitations, and testing strategy.
  • Adds example CLI outputs and a quick command reference appendix.

Comment thread doc/sfputil/fw-mgmt-enhancement.md
Comment thread doc/sfputil/fw-mgmt-enhancement.md
Comment thread doc/sfputil/fw-mgmt-enhancement.md
Comment thread doc/sfputil/fw-mgmt-enhancement.md Outdated
Comment thread doc/sfputil/fw-mgmt-enhancement.md Outdated
Comment thread doc/sfputil/fw-mgmt-enhancement.md Outdated
Comment thread doc/sfputil/fw-mgmt-enhancement.md Outdated
Comment thread doc/sfputil/fw-mgmt-enhancement.md Outdated
Comment thread doc/sfputil/fw-mgmt-enhancement.md Outdated
Comment thread doc/sfputil/fw-mgmt-enhancement.md Outdated
Signed-off-by: Rohit Sharma <rohit@nexthop.ai>
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.

5 participants