Skip to content

[Overlay] Add BGP profile to Vnet routes#2337

Merged
prsunny merged 1 commit intosonic-net:masterfrom
prsunny:profile
Jul 8, 2022
Merged

[Overlay] Add BGP profile to Vnet routes#2337
prsunny merged 1 commit intosonic-net:masterfrom
prsunny:profile

Conversation

@prsunny
Copy link
Copy Markdown
Collaborator

@prsunny prsunny commented Jun 18, 2022

What I did
Add BGP profile support to Vnet routes

Why I did it
If a profile is specified in Vnet routes, it must be reflected in the STATE_DB routes for advertising:

HGETALL "ADVERTISE_NETWORK_TABLE|150.62.192.1/32"
1) "profile"
2) "FROM_SLB_ROUTES"

How I verified it
With VS tests

Details if related

@prsunny prsunny requested a review from sumukhatv June 20, 2022 16:57
@prsunny prsunny marked this pull request as ready for review June 20, 2022 16:58
@prsunny
Copy link
Copy Markdown
Collaborator Author

prsunny commented Jun 20, 2022

@StormLiangMS for viz

@prsunny
Copy link
Copy Markdown
Collaborator Author

prsunny commented Jun 20, 2022

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Copy Markdown
Collaborator Author

prsunny commented Jun 27, 2022

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Copy Markdown
Collaborator Author

prsunny commented Jul 8, 2022

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny prsunny merged commit bf91a49 into sonic-net:master Jul 8, 2022
@prsunny prsunny deleted the profile branch July 8, 2022 22:22
yxieca pushed a commit that referenced this pull request Jul 17, 2022
*Add BGP profile support to Vnet routes
preetham-singh pushed a commit to preetham-singh/sonic-swss that referenced this pull request Aug 6, 2022
*Add BGP profile support to Vnet routes
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
*Add BGP profile support to Vnet routes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants