Skip to content
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

[sonic-sairedis] Update Submodule #5052

Merged
merged 1 commit into from
Jul 28, 2020
Merged

[sonic-sairedis] Update Submodule #5052

merged 1 commit into from
Jul 28, 2020

Conversation

gechiang
Copy link
Collaborator

Advance submodule head for sonic-sairedis on Master branch

- Why I did it
The following are the commits in the submodule:

Workaround getSwitchType failure due to NPU SAI not yet supporting it (#647)
commit 4a3a568ad12d8f953c4b4a33ae39744a5bd57025

Adding support for new SAI API sai_query_attribute_capability() (#645)
commit 8d3ae8c6546dcfdc06cf84e1a59bc2a615651a72

[syncd] ARM32 bit fixes, for 64bit printf format specifier. (#643)  
Signed-off-by: Rajkumar Pennadam Ramamoorthy <rpennadamram@marvell.com>
commit 8809d53871169775e8869328679987ba56e4c426

[syncd] Add support for synchronous bulk api (#640)
commit d54bfb45435685059578ea53453e797b87241617

- How I did it
Update sairedis submodule commit pointer

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@gechiang gechiang requested a review from lguohan July 28, 2020 00:38
@lguohan
Copy link
Collaborator

lguohan commented Jul 28, 2020

retest vsimage please

@gechiang
Copy link
Collaborator Author

gechiang commented Jul 28, 2020

Retested the BRCM image.
Summary:
DUT boot up fine. All services are up. No core/crash observed with the associated changes. Syslog shows the get switch type failed but default to NPU type and moved on. performed "show interface status" and observed all ports/LAGs were up.
I have copied those output in a file and attached
updateSairedisSubmoduleValidationResult.txt

@gechiang gechiang requested a review from lguohan July 28, 2020 15:27
@lguohan
Copy link
Collaborator

lguohan commented Jul 28, 2020

retest vsimage please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants