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

bgpd: route map changes not propagated to existing static network (ipv4 unicast tested) #1117

Closed
louberger opened this issue Sep 7, 2017 · 3 comments

Comments

@louberger
Copy link
Member

louberger commented Sep 7, 2017

  1. add route map, e.g.

route-map rm-nh permit 10
match ip address al-any
set ip next-hop 192.1.1.5
set local-preference 123
set metric 98

  1. advertise static network, e.g.

address-family ipv4 unicast
network 5.0.0.0/24 route-map rm-nh

  1. modify route map, e.g.,

route-map rm-nh permit 10
set large-community 12:34:56
set extcommunity rt 89:123
set community 0:67

  1. add new static network, e.g.

address-family ipv4 unicast
network 5.0.10.0/24 route-map rm-nh

  1. wait some time (minutes)

  2. see that only new static net is impacted by route map change

CE-192.1.1.5# show bgp ipv4 uni 5.0.0.0/24
BGP routing table entry for 5.0.0.0/24
Paths: (1 available, best #1, table Default-IP-Routing-Table)
Advertised to non peer-group peers:
192.1.1.3
Local
192.1.1.5 from 0.0.0.0 (192.1.1.5)
Origin IGP, metric 98, localpref 123, weight 32768, valid, sourced, local, best
AddPath ID: RX 0, TX 2
Last update: Thu Sep 7 09:15:21 2017

CE-192.1.1.5# show bgp ipv4 uni 5.0.10.0/24
BGP routing table entry for 5.0.10.0/24
Paths: (1 available, best #1, table Default-IP-Routing-Table)
Advertised to non peer-group peers:
192.1.1.3
Local
192.1.1.5 from 0.0.0.0 (192.1.1.5)
Origin IGP, metric 98, localpref 123, weight 32768, valid, sourced, local, best
Community: 0:67
Extended Community: RT:89:123
Large Community: 12:34:56
AddPath ID: RX 0, TX 12
Last update: Thu Sep 7 09:16:24 2017

@dwalton76
Copy link
Contributor

I tried this out and it only breaks when --enable-bgp-vnc=yes is used. I don't know the VNC code at all so assigning this to @louberger

@dwalton76 dwalton76 assigned louberger and unassigned dwalton76 Sep 28, 2017
@patrick7
Copy link

patrick7 commented Nov 4, 2017

I'm seeing the same issue

@louberger louberger removed their assignment Jun 5, 2018
@ton31337
Copy link
Member

ton31337 commented Jul 9, 2019

Tested with:

spine1-debian-9# show version 
FRRouting 7.2-dev-MyOwnFRRVersion-g5d5e3d9 (spine1-debian-9).
Copyright 1996-2005 Kunihiro Ishiguro, et al.
This is a git build of frr-7.1-dev-555-gf85d5e3d9
Associated branch(es):
	local:master
	github/ton31337/frr/master

configured with:
    '--enable-systemd' '--enable-exampledir=/usr/share/doc/frr/examples/' '--localstatedir=/var/opt/frr' '--sbindir=/usr/lib/frr' '--sysconfdir=/etc/frr' '--enable-vtysh' '--enable-isisd' '--enable-pimd' '--enable-watchfrr' '--enable-ospfclient=yes' '--enable-ospfapi=yes' '--enable-multipath=64' '--enable-user=frr' '--enable-group=frr' '--enable-vty-group=frrvty' '--enable-configfile-mask=0640' '--enable-logfile-mask=0640' '--enable-rtadv' '--enable-fpm' '--enable-ldpd' '--with-pkg-git-version' '--with-pkg-extra-version=-MyOwnFRRVersion' '--disable-nhrpd' '--enable-bgp-vnc=yes'

Can't reproduce this.

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

No branches or pull requests

4 participants