Skip to content

Switch Capability support#1525

Open
PrabhuSreenivasan wants to merge 3 commits intosonic-net:masterfrom
PrabhuSreenivasan:switch_capability
Open

Switch Capability support#1525
PrabhuSreenivasan wants to merge 3 commits intosonic-net:masterfrom
PrabhuSreenivasan:switch_capability

Conversation

@PrabhuSreenivasan
Copy link
Copy Markdown
Contributor

@PrabhuSreenivasan PrabhuSreenivasan commented Dec 4, 2020

Signed-off-by: Prabhu Sreenivasan [email protected]

What I did
Added switch capability support where Orchagent queries the switch for the list of supported objects (SAI_SWITCH_ATTR_SUPPORTED_OBJECT_TYPE_LIST) and identifies the capabilities / features supported by the switch. It further updates the SWITCH_TABLE:switch on APPL_DB with supported capabilities

eg:

sonic-db-cli APPL_DB hgetall "SWITCH_TABLE:switch"
threshold_supported
True

Why I did it
This allows applications/users to understand whether a feature is supported on a device / SAI implementation.

How I verified it
Query SWITCH_TABLE:switch on APPL_DB for "threshold_supported" entry.

Related PRs
sonic-net/sonic-sairedis#728

Details if related

@PrabhuSreenivasan
Copy link
Copy Markdown
Contributor Author

The vslib support for the same is on sonic-net/sonic-sairedis#728.

@PrabhuSreenivasan
Copy link
Copy Markdown
Contributor Author

retest vs please

@PrabhuSreenivasan
Copy link
Copy Markdown
Contributor Author

@lguohan, Can you please add Label:brcm-upstream to this PR

Signed-off-by: Prabhu Sreenivasan <[email protected]>
@PrabhuSreenivasan
Copy link
Copy Markdown
Contributor Author

How can I fix the below failure? It seems to be coming on latest swss builds

@azure-pipelinesazure-pipelines
/ Azure.sonic-swss
Build log #L12
The container 9968994 could not be found.

@prsunny prsunny self-requested a review as a code owner September 2, 2022 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants