-
Notifications
You must be signed in to change notification settings - Fork 48
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
Integrate ZMQ to GNMI to improve Dash GNMI API performance. #135
Conversation
4186b0b
to
c3a66bc
Compare
/azp run |
No commit pushedDate could be found for PR 135 in repo sonic-net/sonic-gnmi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
/easycla |
/azp run EasyCLA |
No commit pushedDate could be found for PR 135 in repo sonic-net/sonic-gnmi |
EasyCLA not run because a known issue: communitybridge/easycla#4052 |
/easycla |
PR validation failed because no disk space on agent, try close and open PR. |
Integrate ZMQ to GNMI to improve Dash GNMI API performance.
Why I did it
Currently Dash gnmi to orchagent communitcation using redis based channel, switch to ZMQ based change will improve performance.
How I did it
Integrate ZMQ to gnmi service.
How to verify it
Manually test.
Add new UT.
Work item tracking
Microsoft ADO (number only): 17753835
Which release branch to backport (provide reason below if selected)
Description for the changelog
Integrate ZMQ to GNMI to improve Dash GNMI API performance.
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)