Testing PullReq against Topology Tests

Build: #385 failed Manual run by Martin Winter

Build result summary

Details

Completed
Duration
45 minutes
Labels
version=frr-6_1-dev-844-gdc079d4f7git=https_//github_com/frrouting/topotests_gitbuildurl=https_//ci1_netdef_org/browse/frr-topopr-cf-385branch=master
Revisions
FRR - Topotests
512010d4e54e38189ca553a20e088614516a70ec 512010d4e54e38189ca553a20e088614516a70ec
FRR - FreeRangeRouting
dc079d4f7a311f5876f84f606cb2cd199783922d dc079d4f7a311f5876f84f606cb2cd199783922d
Total tests
1240
Failing since
#381 (Custom build by Github Hook with revision master)
Fixed in
#391 (Custom build by Github Hook with revision master)

Responsible

Code commits

FRR - Topotests
Author Commit Message Commit date
Martin Winter Martin Winter 512010d4e54e38189ca553a20e088614516a70ec 512010d4e54e38189ca553a20e088614516a70ec Merge pull request #143 from opensourcerouting/bgp-ecmp-default-fix
bgp-ecmp-topo1: fix and improve topology test
Rafael Zalamena Rafael Zalamena 57ed2311568ac6bafe793b979df562ef9338a9cd 57ed2311568ac6bafe793b979df562ef9338a9cd bgp-ecmp-topo1: fix some pylint warnings
Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Rafael Zalamena Rafael Zalamena 5a87c583140e713394739b97b78d96cdd859488b 5a87c583140e713394739b97b78d96cdd859488b bgp-ecmp-topo1: fix vrf default change
Python's `string' method 'replace' doesn't actually modify the string,
instead it returns a new string with the modified content.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
Rafael Zalamena <rzalamena@users.noreply.github.com> Rafael Zalamena <rzalamena@users.noreply.github.com> 132ffd115b89efd6c21aba86f2cd362b61035b2f 132ffd115b89efd6c21aba86f2cd362b61035b2f Merge pull request #138 from pguibert6WIND/movevrfdeftodefault
bgp-ecmp-topo1: change default vrf name
Philippe Guibert Philippe Guibert 53b7916fbf34c6a66e4857d61588dc474fdbce4c 53b7916fbf34c6a66e4857d61588dc474fdbce4c bgp-ecmp-topo1: change default vrf name
By default, default vrf name is default, whereas BGP default vrf was
hardcoded to Default. Because this will be fixes, do not care about the
vrf name field and apply a regexp to move Default to default.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Tests

New test failures 4
Status Test View job Duration
Collapse Failed test_bfd_topo1 test_bgp_convergence History
AddressSanitizer 43 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
AddressSanitizer 1 min
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_bfd_topo1 test_bgp_fast_reconvergence History
AddressSanitizer 13 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_bgp_rfapi_basic_sanity_config2 test_check_timeout History
AddressSanitizer 1 min
AssertionError: 1 tests failed
E   AssertionError: 1 tests failed

Shared artifacts

Artifact File size
VersionInfo 690 bytes
SourceCode 4 MB
topotests 534 KB