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

[bgp][minigraph] Add support for parsing bgp_router_id in minigraph parser #18698

Closed

Conversation

yaqiangz
Copy link
Contributor

@yaqiangz yaqiangz commented Apr 17, 2024

Why I did it

Add support for parsing bgp_router_id in minigraph parser
HLD: sonic-net/SONiC#1643

Work item tracking
  • Microsoft ADO (number only): 27706475

How I did it

Add support for parsing bgp_router_id in minigraph parser

How to verify it

UT, Image built successfully

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@yaqiangz yaqiangz force-pushed the master_minigraph_router_id branch 3 times, most recently from 75e827a to 7ce2dcb Compare April 18, 2024 02:43
@yaqiangz yaqiangz marked this pull request as ready for review April 23, 2024 03:46
@yaqiangz yaqiangz requested a review from qiluo-msft as a code owner April 23, 2024 03:46
@yaqiangz yaqiangz force-pushed the master_minigraph_router_id branch from 7ce2dcb to e89628c Compare April 25, 2024 23:44
@lguohan
Copy link
Collaborator

lguohan commented May 13, 2024

@yaqiangz , can you resolve the conflict?

@yaqiangz
Copy link
Contributor Author

@yaqiangz , can you resolve the conflict?

Hi @lguohan I have synced with Qi offline, no need to add this change.

@yaqiangz yaqiangz closed this May 13, 2024
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.

2 participants