Skip to content

[topo] Fix incorrect next hop ips for various topologies#2854

Merged
lguohan merged 1 commit intosonic-net:masterfrom
neethajohn:fix_nh_ips
Jan 26, 2021
Merged

[topo] Fix incorrect next hop ips for various topologies#2854
lguohan merged 1 commit intosonic-net:masterfrom
neethajohn:fix_nh_ips

Conversation

@neethajohn
Copy link
Copy Markdown
Contributor

Signed-off-by: Neetha John nejo@microsoft.com

Description of PR

Update nh ipv4 and ipv6 ips for various topologies

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Approach

What is the motivation for this PR?

Exabgp sessions were not getting setup with the VMs because of incorrect v4 and v6 neighbors

How did you verify/test it?

Verified by running announce_routes on t0-52 topo and all routes were populated on DUT

Signed-off-by: Neetha John <nejo@microsoft.com>
@neethajohn neethajohn requested a review from a team January 23, 2021 01:38
@neethajohn
Copy link
Copy Markdown
Contributor Author

retest this please

1 similar comment
@neethajohn
Copy link
Copy Markdown
Contributor Author

retest this please

@neethajohn
Copy link
Copy Markdown
Contributor Author

retest vsimage please

@daall
Copy link
Copy Markdown
Contributor

daall commented Jan 25, 2021

retest this please

1 similar comment
@daall
Copy link
Copy Markdown
Contributor

daall commented Jan 25, 2021

retest this please

@lguohan lguohan merged commit b2a76ab into sonic-net:master Jan 26, 2021
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…lly (sonic-net#15796)

#### Why I did it
src/sonic-swss
```
* 0ec46f22 - (HEAD -> 202211, origin/202211) [muxorch] Skip programming ACL for standby `active-active` ports (sonic-net#2569) (sonic-net#2854) (7 hours ago) [Longxiang Lyu]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants