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

syncd 202012 branch on SAI bulk API enabling is un-syncd to syncd master branch on processBulkOidRemove #1125

Open
yenlu-keith opened this issue Sep 9, 2022 · 1 comment

Comments

@yenlu-keith
Copy link
Contributor

yenlu-keith commented Sep 9, 2022

[issue]

enabling SAI bulk API on syncd container will produce the SAI ERR msg as below when running with sonic-mgmt suite on ipfwd/test_nhop_count.py
ERR syncd#syncd: SAI_LOG|SAI_API_NEXT_HOP_GROUP: Bad object id 0x2d000000025e90

after reviewing the code azure 202012 branch use VID instead of RID when doing processBulkOidRemove.
no matter Bulk or normal, syncd layer will use RID to SAI layer.

https://github.com/sonic-net/sonic-sairedis/blob/master/syncd/Syncd.cpp#L1607
https://github.com/sonic-net/sonic-sairedis/blob/202012/syncd/Syncd.cpp#L1432

it should be incorrect on 202012 syncd codebase.

[reproduce steps]

enabling SAI bulk API on syncd container by
docker exec -it syncd /bin/bash
vi /usr/bin/syncd_init_common.sh
//Use bulk api`s in SAI
//currently disabled since most vendors don't support that yet
CMD_ARGS+=" -l"

run with sonic-mgmt suite on ipfwd/test_nhop_count.py

@yenlu-keith yenlu-keith changed the title syncd 202012 codes on SAI bulk API enabling is un-syncd to syncd master branch on processBulkOidRemove syncd 202012 branch on SAI bulk API enabling is un-syncd to syncd master branch on processBulkOidRemove Sep 9, 2022
@yenlu-keith
Copy link
Contributor Author

here is the PR commit; please kindly double commit to 202012 branch
#854

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

No branches or pull requests

1 participant