Skip to content

[202205] Resolve neighbors from config_db#15008

Closed
anish-n wants to merge 1 commit intosonic-net:202205from
anish-n:arp_update_202205
Closed

[202205] Resolve neighbors from config_db#15008
anish-n wants to merge 1 commit intosonic-net:202205from
anish-n:arp_update_202205

Conversation

@anish-n
Copy link
Contributor

@anish-n anish-n commented May 10, 2023

Why I did it

To resolve NEIGH table entries present in CONFIG_DB. Without this change arp/ndp entries which we wish to resolve, and configured via CONFIG_DB are not resolved.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Modify arp_update script to take NEIGH entries from config db for resolution. For failed entries trigger a ping6/ping command for resolution

How to verify it

Configure NEIGH table in config_db and check if it gets resolved.

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

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

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)

@anish-n anish-n requested a review from lguohan as a code owner May 10, 2023 18:19
@theasianpianist
Copy link
Contributor

Is there any reason we need a separate PR for the 202205 branch? Might be better to just cherry-pick the master branch PR after it merges.

@prsunny
Copy link
Contributor

prsunny commented May 10, 2023

Added label to master PR - #15006. We can use this if there is a cherry-pick conflict

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