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.

Sign up for GitHub

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

Jump to bottom

NLRI next hops are not formatted properly for ipv4 vpn #71

Closed
louberger opened this issue Jan 13, 2017 · 2 comments
Closed

NLRI next hops are not formatted properly for ipv4 vpn #71

louberger opened this issue Jan 13, 2017 · 2 comments
Milestone
2.0-rc1

Comments

@louberger
Copy link
Member

Seen in wireshark, Works fine in Quagga.
Format is correct when constructed.
Check in debugger @ bgp_updgrp_packet.c line 811 (stable/2.0)
louberger added a commit to LabNConsulting/frr that referenced this issue Jan 15, 2017
@louberger
bgpd: fix RD stomping by update group code (Issue FRRouting#71)
be6eff5 
Signed-off-by: Lou Berger <lberger@labn.net>
louberger added a commit to LabNConsulting/frr that referenced this issue Jan 15, 2017
@louberger
bgpd: allow VPN next hop to be different AFI than NLRI next hop (Issue …
74031c1 
…FRRouting#71)

Signed-off-by: Lou Berger <lberger@labn.net>
@louberger louberger mentioned this issue Jan 16, 2017
Closed
@donaldsharp donaldsharp added this to the 2.0-rc1 milestone Jan 17, 2017
louberger added a commit to LabNConsulting/frr that referenced this issue Jan 17, 2017
@louberger
bgpd: fix RD stomping by update group code (Issue FRRouting#71)
9857daf 
Signed-off-by: Lou Berger <lberger@labn.net>
louberger added a commit to LabNConsulting/frr that referenced this issue Jan 17, 2017
@louberger
bgpd: allow VPN next hop to be different AFI than NLRI next hop (Issue …
4c6f219 
…FRRouting#71)

Signed-off-by: Lou Berger <lberger@labn.net>
louberger added a commit to LabNConsulting/frr that referenced this issue Jan 18, 2017
@louberger
bgpd: fix typo in VPN next hop V6 test (Issue FRRouting#71)
7a4732c 
      On top of:
      bgpd: allow VPN next hop to be different AFI than NLRI next
      	    hop (Issue FRRouting#71)

Signed-off-by: Lou Berger <lberger@labn.net>
louberger added a commit to LabNConsulting/frr that referenced this issue Jan 18, 2017
@louberger
bgpd: fix RD stomping by update group code (Issue FRRouting#71)
e8d6265 
Signed-off-by: Lou Berger <lberger@labn.net>
louberger added a commit to LabNConsulting/frr that referenced this issue Jan 18, 2017
@louberger
bgpd: allow VPN next hop to be different AFI than NLRI next hop (Issue …
fe3ca08 
…FRRouting#71)

Signed-off-by: Lou Berger <lberger@labn.net>
@louberger louberger mentioned this issue Jan 18, 2017
Merged
@louberger
Copy link
Member Author

fixed in #83

@eqvinox
Copy link
Contributor

eqvinox commented Jan 19, 2017

#83 merged - closing.

@eqvinox eqvinox closed this as completed Jan 19, 2017
@pguibert6WIND pguibert6WIND mentioned this issue May 11, 2017
Merged
cfra referenced this issue in opensourcerouting/frr Nov 29, 2018
@mwinter-osr
Merge pull request #71 from LabNConsulting/working/lutil-count+return…
ab79b6f 
…Json

 bgp_l3vpn_to_bgp_vrf improvements
@NetDEF-CI NetDEF-CI mentioned this issue Mar 12, 2021
Merged
@KusakabeShi KusakabeShi mentioned this issue Jul 31, 2021
Closed
gpnaveen pushed a commit to gpnaveen/frr that referenced this issue Oct 29, 2021
@pushpasis
Merge pull request FRRouting#71 from gpnaveen/mgmt_automation
7c9d387 
tests: adding mgmt automation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0-rc1
Development

No branches or pull requests
3 participants
@eqvinox @louberger @donaldsharp