Testing PullReq against Topology Tests

Build: #378 failed Rebuilt by Martin Winter

Build result summary

Details

Completed
Duration
45 minutes
Flags
Custom revisionCustom build
Labels
version=frr-6_1-dev-730-g8dfa7e468git=https_//github_com/frrouting/topotests_gitbuildurl=https_//ci1_netdef_org/browse/frr-topopr-cf-378branch=docker
Revisions
FRR - Topotests
docker docker
FRR - FreeRangeRouting
8dfa7e4680c3d8f2606d7fdef77df0e9cb44c645 8dfa7e4680c3d8f2606d7fdef77df0e9cb44c645
Total tests
1116
Failing since
#376 (Custom build by Github Hook with revision master)
Fixed in
#379 (Rebuilt by Martin Winter)
Number of retries
1

Tests

Custom variables

Custom revision
docker docker
Name Value
github_base_sha 634b1d6f46fefada44b8f147c70e746b18fccf4f
github_pullreq 141
github_branch docker
github_repo FRRouting/topotests
github_merge_sha fd3bd8c1ec17ae3698d5bcc3d6c130606068010a

Comments

  1. Github Hook

    GitHub Merge Request 141
    for GitHub Repo FRRouting/topotests, branch docker

    Request to merge from cfra/topotests
    Merge Git Commit ID fd3bd8c1ec17ae3698d5bcc3d6c130606068010a on top of base Git Commit ID 634b1d6f46fefada44b8f147c70e746b18fccf4f

Code commits

FRR - Topotests
Author Commit Message Commit date
Rafael Zalamena <rzalamena@users.noreply.github.com> Rafael Zalamena <rzalamena@users.noreply.github.com> 634b1d6f46fefada44b8f147c70e746b18fccf4f 634b1d6f46fefada44b8f147c70e746b18fccf4f Merge pull request #137 from cfra/docker
Update docker branch to make it more approachable
FRR - FreeRangeRouting
Author Commit Message Commit date
Renato Westphal <renato@openbsd.org> Renato Westphal <renato@openbsd.org> 8dfa7e4680c3d8f2606d7fdef77df0e9cb44c645 8dfa7e4680c3d8f2606d7fdef77df0e9cb44c645 Merge pull request #3162 from pguibert6WIND/vpn_route_map_issue
bgpd: vpn route-map config should be kept, except if vrf list is on
Renato Westphal <renato@openbsd.org> Renato Westphal <renato@openbsd.org> fb52f65126f6f5a053ac86e8ca871db02e70ab6d fb52f65126f6f5a053ac86e8ca871db02e70ab6d Merge pull request #3271 from donaldsharp/nh_encode_better
lib, zebra: Encode nexthop vrf in nht updates
Donald Sharp Donald Sharp a756969dc61880d781060a9387c7bdf6da02a238 a756969dc61880d781060a9387c7bdf6da02a238 pimd, zebra: Encode nexthop vrf in message
Encode the nexthop vrf in the message sent about mrib lookups.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Donald Sharp Donald Sharp b6c9de3bd3eb750a0ce1fcca66a38416a8d767d0 b6c9de3bd3eb750a0ce1fcca66a38416a8d767d0 lib, zebra: Encode nexthop vrf in nht updates
The nexthop vrf was not being encoded in nht updates.
Add it in.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Tests

New test failures 7
Status Test View job Duration
Collapse Failed test_bfd_topo1 test_bgp_fast_convergence History
AddressSanitizer 1 min
AssertionError: r4: bgp did not converge assert expected key(s) [u'10.254.254.1/32'] in json["routes"] (have [u'10.254.254.4/32', u'10.254.254.3/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.3/32": [   @@ -20 +28,3 @@   -            "aspath": "102",    +            "aspath": "102 103",    +            "bestpath": true,    +            "network": "10.254.254.3/32",    @@ -27,0 +38,3 @@   +            "origin": "IGP",    +            "path": "102 103",    +            "pathFrom": "external",    @@ -29 +42 @@   -            "prefix": "10.254.254.2",    +            "prefix": "10.254.254.3",    @@ -31 +44,2 @@   -            "valid": true   +            "valid": true,    +            "weight": 0   @@ -34 +48 @@   -    "10.254.254.3/32": [   +    "10.254.254.4/32": [   @@ -36 +50,5 @@   -            "aspath": "102 103",    +            "aspath": "",    +            "bestpath": true,    +            "med": 0,    +            "metric": 0,    +            "network": "10.254.254.4/32",    @@ -40 +58 @@   -                    "ip": "192.168.2.2",    +                    "ip": "0.0.0.0",    @@ -44,2 +62,5 @@   -            "peerId": "192.168.2.2",    -            "prefix": "10.254.254.3",    +            "origin": "IGP",    +            "path": "",    +            "pathFrom": "external",    +            "peerId": "(unspec)",    +            "prefix": "10.254.254.4",    @@ -47 +68,2 @@   -            "valid": true   +            "valid": true,    +            "weight": 32768
E   AssertionError: r4: bgp did not converge
    assert expected key(s) [u'10.254.254.1/32'] in json["routes"] (have [u'10.254.254.4/32', u'10.254.254.3/32', u'10.254.254.2/32']):
      --- Expected value
      +++ Current value
      @@ -2 +2 @@
      -    "10.254.254.1/32": [
      +    "10.254.254.2/32": [
(62 more lines...)
Collapse Failed test_bfd_topo1 test_bgp_convergence History
AddressSanitizer 40 secs
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")
Collapse Failed test_bfd_topo1 test_bgp_fast_reconvergence History
TopoTest Master arm8 6 secs
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...)
Collapse Failed test_bfd_topo1 test_bgp_convergence History
TopoTest Master arm8 25 secs
AssertionError: r2: bgp did not converge assert json["ipv4Unicast"]["peers"]["192.168.1.1"]["state"] value is different (   --- Expected value   +++ Current value   @@ -1 +1 @@   -"Established"   +"Idle")
E   AssertionError: r2: bgp did not converge
    assert json["ipv4Unicast"]["peers"]["192.168.1.1"]["state"] value is different (
      --- Expected value
      +++ Current value
      @@ -1 +1 @@
      -"Established"
      +"Idle")
Collapse Failed test_bfd_topo1 test_bgp_fast_convergence History
TopoTest Master arm8 41 secs
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', u'10.254.254.1/32']):   --- Expected value   +++ Current value   @@ -4,0 +5,2 @@   +            "bestpath": true,    +            "network": "10.254.254.1/32",    @@ -11,0 +14,3 @@   +            "origin": "IGP",    +            "path": "102 101",    +            "pathFrom": "external",    @@ -15 +20,2 @@   -            "valid": true   +            "valid": true,    +            "weight": 0   @@ -20,0 +27,4 @@   +            "bestpath": true,    +            "med": 0,    +            "metric": 0,    +            "network": "10.254.254.2/32",    @@ -27,0 +38,3 @@   +            "origin": "IGP",    +            "path": "102",    +            "pathFrom": "external",    @@ -31 +44,2 @@   -            "valid": true   +            "valid": true,    +            "weight": 0   @@ -34 +48 @@   -    "10.254.254.3/32": [   +    "10.254.254.4/32": [   @@ -36 +50,5 @@   -            "aspath": "102 103",    +            "aspath": "",    +            "bestpath": true,    +            "med": 0,    +            "metric": 0,    +            "network": "10.254.254.4/32",    @@ -40 +58 @@   -                    "ip": "192.168.2.2",    +                    "ip": "0.0.0.0",    @@ -44,2 +62,5 @@   -            "peerId": "192.168.2.2",    -            "prefix": "10.254.254.3",    +            "origin": "IGP",    +            "path": "",    +            "pathFrom": "external",    +            "peerId": "(unspec)",    +            "prefix": "10.254.254.4",    @@ -47 +68,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', u'10.254.254.1/32']):
      --- Expected value
      +++ Current value
      @@ -4,0 +5,2 @@
      +            "bestpath": true, 
      +            "network": "10.254.254.1/32", 
(46 more lines...)
Collapse Failed test_bgp_rfapi_basic_sanity_config2 test_check_timeout History
AddressSanitizer 1 min
AssertionError: 1 tests failed
E   AssertionError: 1 tests failed
Collapse Failed test_bgp_rfapi_basic_sanity test_check_timeout History
AddressSanitizer 1 min
AssertionError: 1 tests failed
E   AssertionError: 1 tests failed

Shared artifacts

Artifact File size
VersionInfo 784 bytes
SourceCode 4 MB
topotests 541 KB