-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[sairedis] update submodule for sairedis #3984
Conversation
@kcudnik please help check if it is safe to update submodule including all the sairedis related changes before my multiDB changes, is there any dependency? On local, I saw all the vs tests passed. |
Not sure why vs failed, i dont see clear indication of issues with pointer, errors are related to snmp, i will request rebuild to see if it passes |
retest this please |
retest vsimage please |
1 similar comment
retest vsimage please |
fails every time on
E KeyError: 'ansible_facts' probably this is indirect error related to changes in sairedis :/ |
But if you look at other PRs, they all failed here |
retest vsimage please |
1 similar comment
retest vsimage please |
ahh, so if other failed then seems like this testbed is unstable or some other issue |
retest vsimage please |
build system must be fixed instead of getting lucky on build |
retest vsimage please |
1 similar comment
retest vsimage please |
@kcudnik after the vs image build system fix, this update passed |
update multiDB changes in sairedis, including:
[MultiDB]: repalce old APIs with New APIs incuding testing ([sonic-utilities]: Update submodule #537)
[syncd] Move port map and port map parser to proper class (add an option to specify the baud rate for sonic #542)
[syncd] Update syncd to expect correct object query message (No such file or directory #include <switch_sai_rpc_server.h> #548)
Support for snat available entry switch attribute in vslib (needed to run vs pytests) ([swss]: Update submodule #546)
Move apiInitialized flag to Globals namespace ([thrift]: disable auto test in thrift package build #545)
Move command line options and parser to separate classes ([mellanox]: Update hw-management to V1.0.0070 #541)
Move ntf_queue to proper NotificationQueue class ([database]: Remove hard/soft limits for client-output-buffer in redis #539)
Move api mutex to global class and add sairedis namespace ([device]: Add support for Mellanox MSN2100 #544)
Clean sairedis.h header ([Broadcom]: Update broacom SAI to 2.1.5.1-4 and opennsl to 3.2.2.2-3 #543)
Fix mlnx.pl script for perl include local directory (Update MLNX SAI revision reference #540)
Layer 2 Forwarding Enhancements ([build]: Adding support for Free-Range-Routing stack. #510)
@kcudnik please help check if it is safe to update submodule including all the sairedis related changes before my multiDB changes, is there any dependency? On local, I saw all the vs tests passed.
Signed-off-by: Dong Zhang d.zhang@alibaba-inc.com