Build #2,003

FreeRangeRouting Protocol Suite

Build result summary

Details

Completed
Duration
125 minutes
Labels
version=frr-7_1-dev-530-gf05d88804git=https_//github_com/frrouting/frr_gitbuildurl=https_//ci1_netdef_org/browse/frr-frr-2003branch=master
Revision
f05d8880492312e5ca79dfd6eb174d066ff07eb9 f05d8880492312e5ca79dfd6eb174d066ff07eb9
Total tests
6635
Failing since
#1997 (Rebuilt by Martin Winter)
Fixed in
#2012 (Changes by Quentin Young)

Tests

Responsible

Code commits

Author Commit Message Commit date
Sri Mohana Singamsetty <srimohans@gmail.com> Sri Mohana Singamsetty <srimohans@gmail.com> f05d8880492312e5ca79dfd6eb174d066ff07eb9 f05d8880492312e5ca79dfd6eb174d066ff07eb9 Merge pull request #3892 from vivek-cumulus/evpn_vrf_route_leak
Leaking of EVPN-based IPv4 and IPv6 routes between VRFs
vivek <vivek@cumulusnetworks.com> vivek <vivek@cumulusnetworks.com> 08e68f925e274989c09249449c12f8870604ad5d m 08e68f925e274989c09249449c12f8870604ad5d bgpd: Recursively determine if route's source is EVPN
With leaking of IPv4 or IPv6 unicast routes whose source is a EVPN
type-2 or type-5 route between VRFs, the determination of whether
the route's source is EVPN has to be made recursively. This is used
during route install to pass along appropriate parameters to zebra.

Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Reviewed-by:   Anuradha Karuppiah <anuradhak@cumulusnetworks.com>
Reviewed-by:   Donald Sharp <sharpd@cumulusnetworks.com>
vivek <vivek@cumulusnetworks.com> vivek <vivek@cumulusnetworks.com> 744c63be132a4644c7c3799a0f0d2a07bfc5b98a m 744c63be132a4644c7c3799a0f0d2a07bfc5b98a zebra: Use next hop's VRF for EVPN-based routes
Ensure that the next hop's VRF is used for IPv4 and IPv6 unicast routes
sourced from EVPN routes, for next hop and Router MAC tracking and
install. This way, leaked routes from other instances are handled properly.

Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Reviewed-by:   Anuradha Karuppiah <anuradhak@cumulusnetworks.com>
Reviewed-by:   Donald Sharp <sharpd@cumulusnetworks.com>
vivek <vivek@cumulusnetworks.com> vivek <vivek@cumulusnetworks.com> 9544ddb28842e5e73e86880f3b12fa52bf264817 m 9544ddb28842e5e73e86880f3b12fa52bf264817 bgpd: Correctly identify VPN-imported routes in a VRF
Refine check that looks for VPN routes imported into a VRF because
a VRF can have other imported routes too like IPv4 and IPv6 unicast
routes sourced from EVPN type-2 and type-5 routes.

Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Reviewed-by:   Anuradha Karuppiah <anuradhak@cumulusnetworks.com>
Reviewed-by:   Donald Sharp <sharpd@cumulusnetworks.com>
vivek <vivek@cumulusnetworks.com> vivek <vivek@cumulusnetworks.com> f106e3a72d37173b68f3f253d70a4af4ab7a7726 m f106e3a72d37173b68f3f253d70a4af4ab7a7726 bgpd: Allow EVPN-sourced routes to be leaked back into EVPN
Refine check on whether a route can be injected into EVPN to allow
EVPN-sourced routes to be injected back into another instance.

Signed-off-by: Vivek Venkatraman <vivek@cumulusnetworks.com>
Reviewed-by:   Anuradha Karuppiah <anuradhak@cumulusnetworks.com>
Reviewed-by:   Donald Sharp <sharpd@cumulusnetworks.com>

Tests

Existing test failures 3
Status Test Failing since View job Duration
AssertionError: r1: bgp did not converge assert json["ipv4Unicast"]["peers"]["192.168.0.2"]["state"] value is different (   --- Expected value   +++ Current value   @@ -1 +1 @@   -"Established"   +"Idle")
E   AssertionError: r1: bgp did not converge
    assert json["ipv4Unicast"]["peers"]["192.168.0.2"]["state"] value is different (
      --- Expected value
      +++ Current value
      @@ -1 +1 @@
      -"Established"
      +"Idle")
AssertionError: r4: bgp did not converge assert expected key(s) [u'10.254.254.3/32', u'10.254.254.1/32'] in json["routes"] (have [u'10.254.254.4/32', u'10.254.254.2/32']):   --- Expected value   +++ Current value   @@ -2 +2 @@   -    "10.254.254.1/32": [   +    "10.254.254.2/32": [   @@ -4 +4,5 @@   -            "aspath": "102 101",    +            "aspath": "102",    +            "bestpath": true,    +            "med": 0,    +            "metric": 0,    +            "network": "10.254.254.2/32",    @@ -11,0 +16,3 @@   +            "origin": "IGP",    +            "path": "102",    +            "pathFrom": "external",    @@ -13 +20 @@   -            "prefix": "10.254.254.1",    +            "prefix": "10.254.254.2",    @@ -15 +22,2 @@   -            "valid": true   +            "valid": true,    +            "weight": 0   @@ -18 +26 @@   -    "10.254.254.2/32": [   +    "10.254.254.4/32": [   @@ -20 +28,5 @@   -            "aspath": "102",    +            "aspath": "",    +            "bestpath": true,    +            "med": 0,    +            "metric": 0,    +            "network": "10.254.254.4/32",    @@ -24 +36 @@   -                    "ip": "192.168.2.2",    +                    "ip": "0.0.0.0",    @@ -28,2 +40,5 @@   -            "peerId": "192.168.2.2",    -            "prefix": "10.254.254.2",    +            "origin": "IGP",    +            "path": "",    +            "pathFrom": "external",    +            "peerId": "(unspec)",    +            "prefix": "10.254.254.4",    @@ -31,17 +46,2 @@   -            "valid": true   -        }   -    ],    -    "10.254.254.3/32": [   -        {   -            "aspath": "102 103",    -            "nexthops": [   -                {   -                    "afi": "ipv4",    -                    "ip": "192.168.2.2",    -                    "used": true   -                }   -            ],    -            "peerId": "192.168.2.2",    -            "prefix": "10.254.254.3",    -            "prefixLen": 32,    -            "valid": true   +            "valid": true,    +            "weight": 32768
E   AssertionError: r4: bgp did not converge
    assert expected key(s) [u'10.254.254.3/32', u'10.254.254.1/32'] in json["routes"] (have [u'10.254.254.4/32', u'10.254.254.2/32']):
      --- Expected value
      +++ Current value
      @@ -2 +2 @@
      -    "10.254.254.1/32": [
      +    "10.254.254.2/32": [
(59 more lines...)
AssertionError: r4: bgp did not converge assert expected key(s) [u'10.254.254.3/32'] in json["routes"] (have [u'10.254.254.4/32', u'10.254.254.2/32']):   --- Expected value   +++ Current value   @@ -2 +1,0 @@   -    "10.254.254.1/32": null,    @@ -5,0 +5,4 @@   +            "bestpath": true,    +            "med": 0,    +            "metric": 0,    +            "network": "10.254.254.2/32",    @@ -12,0 +16,3 @@   +            "origin": "IGP",    +            "path": "102",    +            "pathFrom": "external",    @@ -16 +22,2 @@   -            "valid": true   +            "valid": true,    +            "weight": 0   @@ -19 +26 @@   -    "10.254.254.3/32": [   +    "10.254.254.4/32": [   @@ -21 +28,5 @@   -            "aspath": "102 103",    +            "aspath": "",    +            "bestpath": true,    +            "med": 0,    +            "metric": 0,    +            "network": "10.254.254.4/32",    @@ -25 +36 @@   -                    "ip": "192.168.2.2",    +                    "ip": "0.0.0.0",    @@ -29,2 +40,5 @@   -            "peerId": "192.168.2.2",    -            "prefix": "10.254.254.3",    +            "origin": "IGP",    +            "path": "",    +            "pathFrom": "external",    +            "peerId": "(unspec)",    +            "prefix": "10.254.254.4",    @@ -32 +46,2 @@   -            "valid": true   +            "valid": true,    +            "weight": 32768
E   AssertionError: r4: bgp did not converge
    assert expected key(s) [u'10.254.254.3/32'] in json["routes"] (have [u'10.254.254.4/32', u'10.254.254.2/32']):
      --- Expected value
      +++ Current value
      @@ -2 +1,0 @@
      -    "10.254.254.1/32": null, 
      @@ -5,0 +5,4 @@
(37 more lines...)