-
Notifications
You must be signed in to change notification settings - Fork 549
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ignore failure in setPortPfcAsym if SAI_STATUS_NOT_SUPPORTED #3430
Conversation
Signed-off-by: Zhixin Zhu <[email protected]>
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
…ill make doPortTask keep repeating Signed-off-by: Zhixin Zhu <[email protected]>
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@kperumalbfn @kevinskwang @prsunny This PR is to resolve orchagent crash when there are some ports not supporting PFC. Please help review, thanks. |
Signed-off-by: Zhixin Zhu <[email protected]>
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
@kperumalbfn and @prsunny FYI |
@prsunny This PR is required in 202405 branch, could you help add label for it? |
@zhixzhu Do you need this in 202411 branch? |
@zhixzhu @alpeshspatel With this change, I think we could close the sonic-buildimage PR. |
@zhixzhu @alpeshspatel I see the PR sonic-net/sonic-buildimage#20914 is already closed. |
@kperumalbfn Yes, also need it in 202411 branch. |
@kperumalbfn gentle reminder to check on the merge Labels added to this PR |
Cherry-pick PR to 202411: #3471 |
@kperumalbfn still need it in 202405 branch. |
@bingwang-ms , can you mark this 'approved for 202405'? |
Cherry-pick PR to 202405: #3475 |
3 similar comments
…et#3430) * ignore failure in setPortPfcAsym if SAI_STATUS_NOT_SUPPORTED * return true on setPortPfcAsym SAI_STATUS_NOT_SUPPORTED, since false will make doPortTask keep repeating
What I did
Ignore setPortPfcAsym failure if SAI_STATUS_NOT_SUPPORTED.
Why I did it
Orchagent crash seen.
Some ports don't support PFC feature, if set pfc_asym to off (means SAI_PORT_PRIORITY_FLOW_CONTROL_MODE_COMBINED), it will return SAI_STATUS_NOT_SUPPORTED.
How I verified it
Verified it in testbed. ERR log is kept but no crash.
log:
Details if related