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

SRV6: Add APP_DB tables for SRV6 #538

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

kperumalbfn
Copy link

  • New APP_DB table for SRV6 SID LISt
  • New APP_DB table for SRV6 MY_SID

@@ -108,6 +108,9 @@ namespace swss {

#define APP_NEIGH_RESOLVE_TABLE_NAME "NEIGH_RESOLVE_TABLE"

#define APP_SRV6_SID_LIST_TABLE_NAME "SRV6_SID_LIST_TABLE"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about the Config_DB tables as defined in HLD?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prsunny Since we are going with directly updating the APP_DB for the phase-1 implementation, CONFIG_DB tables and Yang models are not added for the phase-1. Should we add a note about this in HLD?

@zhangyanzhao
Copy link

@caizhenghui-juniper would you please help to review?

@prsunny prsunny merged commit da7a037 into sonic-net:master Oct 19, 2021
@hzheng5 hzheng5 mentioned this pull request Oct 21, 2021
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.

3 participants