test bulk api for vs tests #692
Conversation
Signed-off-by: Guohan Lu <lguohan@gmail.com>
|
@kcudnik, i think the build api for syncd has some issue to pass the vs tests. can someone take a look? |
|
We recently talked that there is and issue and we want to disable "-l" flag, so why enable it in this pr ? just to test it ? |
|
@dmytroxshevchuk you added bulk api support can you take a look why this is failing ? |
|
@lguohan @kcudnik I added bulk API in syncd. As for me my mistake was with log level in case when bulk api is not implemented(its cause of much logs from syncd), for now it is fixed in PR #687, thanks for this. Regarding vs tests it was passed in my PR #656, when bulk was implemented in syncd, so its interesting why tests failing now. |
|
i do not know if it makes difference, but sync mode is now enabled by default on swss |
|
WIP: #694 |
|
this could be revisited, we already have fallback for bulk api in syncd even if vendors don't support that |
this is to test vs tests for the bulk api implementation. DO NOT merge this pr.
Signed-off-by: Guohan Lu lguohan@gmail.com