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

[202012] Advertise ipv6 link local address #1811

Merged
merged 2 commits into from
Sep 15, 2021

Conversation

sumukhatv
Copy link
Contributor

@sumukhatv sumukhatv commented Sep 9, 2021

What I did

Modify the neighbor_advertiser script to advertise ipv6 link local address
This PR resolves sonic-net/sonic-buildimage#6599

Master PRs:

  1. Advertise ipv6 link local address #1402
  2. Refactor neighbor_advertiser script and use mock for testing #1427

How I did it

Add ipv6 link local address to the slice returned by the neighbor advertiser script

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

@qiluo-msft
Copy link
Contributor

Could you make the PR title readable?

@lguohan
Copy link
Contributor

lguohan commented Sep 15, 2021

can you add better description?

@sumukhatv sumukhatv changed the title Neigh adv 202012 Advertise ipv6 link local address Sep 15, 2021
@sumukhatv sumukhatv closed this Sep 15, 2021
@sumukhatv sumukhatv reopened this Sep 15, 2021
@prsunny prsunny changed the title Advertise ipv6 link local address [202012] Advertise ipv6 link local address Sep 15, 2021
@prsunny prsunny merged commit 5323e9d into sonic-net:202012 Sep 15, 2021
stepanblyschak pushed a commit to stepanblyschak/sonic-utilities that referenced this pull request Apr 18, 2022
912d443 [sonic-utilities]remove db_migrator logic for autoneg enable/disable (sonic-net#1823)
5323e9d [202012] Advertise ipv6 link local address (sonic-net#1811)

Signed-off-by: vaibhav-dahiya <vdahiya@microsoft.com>
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.

4 participants