{"action":"created","issue":{"url":"https://api.github.com/repos/FRRouting/frr/issues/1052","repository_url":"https://api.github.com/repos/FRRouting/frr","labels_url":"https://api.github.com/repos/FRRouting/frr/issues/1052/labels{/name}","comments_url":"https://api.github.com/repos/FRRouting/frr/issues/1052/comments","events_url":"https://api.github.com/repos/FRRouting/frr/issues/1052/events","html_url":"https://github.com/FRRouting/frr/issues/1052","id":253108161,"number":1052,"title":"VPN route not always propagated to peer in LabN BGPD CI test","user":{"login":"louberger","id":6575816,"avatar_url":"https://avatars3.githubusercontent.com/u/6575816?v=4","gravatar_id":"","url":"https://api.github.com/users/louberger","html_url":"https://github.com/louberger","followers_url":"https://api.github.com/users/louberger/followers","following_url":"https://api.github.com/users/louberger/following{/other_user}","gists_url":"https://api.github.com/users/louberger/gists{/gist_id}","starred_url":"https://api.github.com/users/louberger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/louberger/subscriptions","organizations_url":"https://api.github.com/users/louberger/orgs","repos_url":"https://api.github.com/users/louberger/repos","events_url":"https://api.github.com/users/louberger/events{/privacy}","received_events_url":"https://api.github.com/users/louberger/received_events","type":"User","site_admin":false},"labels":[{"id":501651398,"url":"https://api.github.com/repos/FRRouting/frr/labels/bug","name":"bug","color":"ee0701","default":true}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-08-26T17:01:46Z","updated_at":"2017-08-28T01:04:48Z","closed_at":null,"body":"First seen on pull request #1023. as stated there, the speculation is that this was actually introduced by PR   #939 .\r\n\r\nTopology is CE4<--(bgp unicast safi)-->PE/GW2<--(vpn safi)-->NVA0<--(vpn safi) -->PE/GW3<--(unicast safi) -->CE5\r\n\r\no Scenario:\r\n1. Add 10 static routes CE4\r\n2. Add VNC route NVA0\r\n3. Confirm NVA0 sees GW2 imported CE4 routes (over vpn safi) and CE5 sees 10 unicast routes\r\n4. Withdraw static routes at CE4\r\n5. Confirm NVA0 see withdrawals\r\n6. Readd static route at CE4\r\n7. Confirm NVA0 see routes\r\n8. Remove VNC route at NVA0\r\n9. See all imported routes removed at NVA0 and GW2\r\n10. Remove static routes\r\n\r\n11. (reverse order of imported routes) Add VNC route NVA0\r\n12. Add 10 static routes CE4\r\n13. Confirm GW 2 sees 10 unicast routes and 11 VPN routes (10 GW2 imported + NVA0 learned)\r\n14. Confirm NVA0 sees  11 VPN routes (10 GW2 imported + NVA0 learned)\r\n\r\nIntermittent failure is on step 14\r\n \r\no Data showing issue: \r\n\r\n-  CE has 10 static routes: (as expected)\r\n>  address-family ipv4 unicast\r\n>   network 5.0.0.0/24 route-map rm-nh\r\n>   network 5.0.1.0/24 route-map rm-nh\r\n>   network 5.0.2.0/24 route-map rm-nh\r\n>   network 5.0.3.0/24 route-map rm-nh\r\n>   network 5.0.4.0/24 route-map rm-nh\r\n>   network 5.0.5.0/24 route-map rm-nh\r\n>   network 5.0.6.0/24 route-map rm-nh\r\n>   network 5.0.7.0/24 route-map rm-nh\r\n>   network 5.0.8.0/24 route-map rm-nh\r\n>   network 5.0.9.0/24 route-map rm-nh\r\n> \r\n> route-map rm-nh permit 10\r\n>  match ip address al-any\r\n>  set ip next-hop 127.0.0.134\r\n>  set local-preference 123\r\n>  set metric 98\r\n\r\n\r\n- GW2 sees 10 routes and imports them to VPN safi, also sees VNC registered route: (as expected)\r\n> GW-127.0.0.132> show bgp ipv4 unicast\r\n> BGP table version is 50, local router ID is 127.0.0.132\r\n> Status codes: s suppressed, d damped, h history, * valid, > best, = multipath,\r\n>               i internal, r RIB-failure, S Stale, R Removed\r\n> Origin codes: i - IGP, e - EGP, ? - incomplete\r\n> \r\n>    Network          Next Hop            Metric LocPrf Weight Path\r\n> *>i5.0.0.0/24       127.0.0.134             98    123      0 i\r\n> *>i5.0.1.0/24       127.0.0.134             98    123      0 i\r\n> *>i5.0.2.0/24       127.0.0.134             98    123      0 i\r\n> *>i5.0.3.0/24       127.0.0.134             98    123      0 i\r\n> *>i5.0.4.0/24       127.0.0.134             98    123      0 i\r\n> *>i5.0.5.0/24       127.0.0.134             98    123      0 i\r\n> *>i5.0.6.0/24       127.0.0.134             98    123      0 i\r\n> *>i5.0.7.0/24       127.0.0.134             98    123      0 i\r\n> *>i5.0.8.0/24       127.0.0.134             98    123      0 i\r\n> *>i5.0.9.0/24       127.0.0.134             98    123      0 i\r\n> \r\n> Displayed  10 routes and 10 total paths\r\n> GW-127.0.0.132> show bgp ipv4 vpn\r\n> BGP table version is 0, local router ID is 127.0.0.132\r\n> Status codes: s suppressed, d damped, h history, * valid, > best, i - internal\r\n> Origin codes: i - IGP, e - EGP, ? - incomplete\r\n> \r\n>    Network          Next Hop            Metric LocPrf Weight Path\r\n> Route Distinguisher: 172.16.1.1:5226\r\n> *>i5.0.0.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> *>i5.0.1.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> *>i5.0.2.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> *>i5.0.3.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> *>i5.0.4.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> *>i5.0.5.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> *>i5.0.6.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> *>i5.0.7.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> *>i5.0.8.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> * i5.0.9.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 127.0.0.134:5226 1000:1 1000:2 ET:7} label=3 life=500 type=bgp-direct, subtype=3\r\n> *>i127.0.0.134/32   172.16.1.1              205      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> \r\n> Displayed 11 routes and 21 total paths\r\n\r\n\r\n- NVA0 only sees 9 of the 10 imported routes: (failure at step 14)\r\n> NVA-127.0.0.130> show bgp ipv4 vpn\r\n> BGP table version is 0, local router ID is 127.0.0.130\r\n> Status codes: s suppressed, d damped, h history, * valid, > best, i - internal\r\n> Origin codes: i - IGP, e - EGP, ? - incomplete\r\n> \r\n>    Network          Next Hop            Metric LocPrf Weight Path\r\n> Route Distinguisher: 172.16.1.1:5226\r\n> *>i5.0.0.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 127.0.0.134:5226 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> *>i5.0.1.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 127.0.0.134:5226 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> *>i5.0.2.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 127.0.0.134:5226 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> *>i5.0.3.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 127.0.0.134:5226 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> *>i5.0.4.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 127.0.0.134:5226 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> *>i5.0.5.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 127.0.0.134:5226 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> *>i5.0.6.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 127.0.0.134:5226 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> *>i5.0.7.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 127.0.0.134:5226 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> *>i5.0.8.0/24       172.16.1.1        98    123      0 ?\r\n>     UN=9.8.7.6 EC{1000:1 1000:2 127.0.0.134:5226 ET:7} label=3 life=500 type=bgp, subtype=0\r\n> *> 127.0.0.134/32   172.16.1.1              205      0 ?\r\n>     UN=9.8.7.6 EC{ET:7 1000:1 1000:2} label=3 life=500 type=bgp, subtype=4\r\n> \r\n> Displayed 10 routes and 19 total paths\r\n"},"comment":{"url":"https://api.github.com/repos/FRRouting/frr/issues/comments/325237417","html_url":"https://github.com/FRRouting/frr/issues/1052#issuecomment-325237417","issue_url":"https://api.github.com/repos/FRRouting/frr/issues/1052","id":325237417,"user":{"login":"louberger","id":6575816,"avatar_url":"https://avatars3.githubusercontent.com/u/6575816?v=4","gravatar_id":"","url":"https://api.github.com/users/louberger","html_url":"https://github.com/louberger","followers_url":"https://api.github.com/users/louberger/followers","following_url":"https://api.github.com/users/louberger/following{/other_user}","gists_url":"https://api.github.com/users/louberger/gists{/gist_id}","starred_url":"https://api.github.com/users/louberger/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/louberger/subscriptions","organizations_url":"https://api.github.com/users/louberger/orgs","repos_url":"https://api.github.com/users/louberger/repos","events_url":"https://api.github.com/users/louberger/events{/privacy}","received_events_url":"https://api.github.com/users/louberger/received_events","type":"User","site_admin":false},"created_at":"2017-08-28T01:04:48Z","updated_at":"2017-08-28T01:04:48Z","body":"Able to reproduce this about 7% of the time on master (c3f779dc1). (18 out of 244)\r\n\r\nAfter 163 runs unable to reproduce on master prior to pull request #939 merge (commit 57a553fd9)\r\n\r\n"}}
{"actor":{"display_login":"louberger"}}