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

Add redis support. #13

Closed
wants to merge 4 commits into from

Conversation

FengPan-Frank
Copy link
Collaborator

@FengPan-Frank FengPan-Frank commented Nov 8, 2024

original reviewed on #7, switch to this PR due to pipeline issue.

Work item tracking
Microsoft ADO (number only):27184382

Make openbmp support redis subscription/population on sonic.

This PR contains below changes:

Use -DENABLE_REDIS=ON to make openbmp support sonic redis population, without -DENABLE_REDIS=ON it keeps its original openbmp functionality/dependency, both of two path verification have been covered by current azure pipeline.
Support BMP_STATE_DB specific table population per users' config via CONFIG_DB, config cli.
Work together with bmpcfgd daemon sonic-net/sonic-host-services#121, and the detail workflow is as below:
3.1 bmpcfgd will be daemon which subscribe config_db table enablement, like bgp_neighbor_table/bgp_rib_in_table/bgp_rib_out_table, once config is changed, bmpcfgd will stop openbmpd, clear bmp related tables, then restart openbmpd
3.2 openbmpd will be daemon which accept connection from bgp, and populate BMP_STATE_DB for relevant bmp table like BGP_NEIGHBOR, etc.
Integrate with swss-common library.

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

Successfully merging this pull request may close these issues.

1 participant