Skip to content

[202511_azd] Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION - #282

Open
longhuan-cisco wants to merge 1 commit into
Azure:202511_azdfrom
longhuan-cisco:longhuan/support_custom_serdes_202511_azd
Open

[202511_azd] Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION#282
longhuan-cisco wants to merge 1 commit into
Azure:202511_azdfrom
longhuan-cisco:longhuan/support_custom_serdes_202511_azd

Conversation

@longhuan-cisco

@longhuan-cisco longhuan-cisco commented Aug 3, 2026

Copy link
Copy Markdown

Cherry-pick of sonic-net/sonic-swss#3764, needed for Cisco 8223.

Companion sonic-platform-daemons cherry-pick PR: Azure/sonic-platform-daemons.msft#80

202511_azd compatibility adjustments:

No additional production behavior is introduced.

@azure-pipelines

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

@longhuan-cisco longhuan-cisco changed the title [202511_azd] Support custom SerDes attributes for Cisco 8223 [202511_azd] Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION Aug 3, 2026
sonic-platform-daemon side change: sonic-net/sonic-platform-daemons#643

What I did
Support SAI_PORT_SERDES_ATTR_CUSTOM_COLLECTION (i.e. custom serdes attributes represented in JSON based string format)

Use boost::variant for the serdes_attr map value to support both std::vector<uint32_t> and std::string in a type-safe way, with easy extensibility to add more types later.
@longhuan-cisco
longhuan-cisco force-pushed the longhuan/support_custom_serdes_202511_azd branch from 4435133 to 34e0f18 Compare August 3, 2026 22:44
@longhuan-cisco

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree company="Cisco"

@longhuan-cisco
longhuan-cisco marked this pull request as ready for review August 3, 2026 23:12
@azure-pipelines

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

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.

1 participant