Testing PullReq against Topology Tests

Build: #47 failed Custom build by Github Hook with revision master

Build result summary

Details

Completed
Duration
35 minutes
Flags
Custom revisionCustom build
Labels
branch=masterbuildurl=https_//ci1_netdef_org/browse/frr-topopr-cf-47git=https_//github_com/frrouting/topotests/pull/36version=frr-3_0-rc1-1860-gb1cebe20
Revisions
FRR - Topotests
master master
FRR - FreeRangeRouting
b1cebe20cd1b5663cc2a47a99ac94dac046b8610 b1cebe20cd1b5663cc2a47a99ac94dac046b8610
Total tests
249
Fixed in
#48 (Custom build by Github Hook with revision master)

Tests

Custom variables

Custom revision
master master
Name Value
github_base_sha a292f190f5928b5b6fb0e8f2c23a5dc0adcf8b30
github_pullreq 36
github_branch master
github_repo FRRouting/topotests
github_merge_sha 90fa9bad278ba73d7f57e1336a48a696b1a2b584

Comments

  1. Github Hook

    GitHub Merge Request 36
    for GitHub Repo FRRouting/topotests, branch master

    Request to merge from opensourcerouting/topotests
    Merge Git Commit ID 90fa9bad278ba73d7f57e1336a48a696b1a2b584 on top of base Git Commit ID a292f190f5928b5b6fb0e8f2c23a5dc0adcf8b30

Code commits

FRR - Topotests
Author Commit Message Commit date
Martin Winter Martin Winter a292f190f5928b5b6fb0e8f2c23a5dc0adcf8b30 a292f190f5928b5b6fb0e8f2c23a5dc0adcf8b30 Merge pull request #35 from opensourcerouting/skip-eigrp-distance-check
eigrp-topo1: Remove check for EIGRP distance
FRR - FreeRangeRouting
Author Commit Message Commit date
Jafar Al-Gharaibeh <Jafaral@users.noreply.github.com> Jafar Al-Gharaibeh <Jafaral@users.noreply.github.com> b1cebe20cd1b5663cc2a47a99ac94dac046b8610 b1cebe20cd1b5663cc2a47a99ac94dac046b8610 Merge pull request #1157 from donaldsharp/recursive_blackhole
Recursive blackhole resolution
Donald Sharp Donald Sharp 972019ae02b5fc6daa3a04790c4687a7399f367c 972019ae02b5fc6daa3a04790c4687a7399f367c zebra: Free leaked zclient data structures on shutdown
On shutdown we were deleting the linked list that
kept the zclient connections, but we were not
freeing the data pointed at by the link list.

This modification allows the normal cleanup of the
linked list to cleanup the zclient data structure.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Donald Sharp Donald Sharp 00c9a53ddc8334e916d64f7f22c092c11ef13531 00c9a53ddc8334e916d64f7f22c092c11ef13531 ripd: Free leaked memory on shutdown
Usage of the address sanitizer and topotests showed
that rip was leaking obuf on shutdown

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Donald Sharp Donald Sharp f637575a9b696f50a3303d0b9e0e5ced7c7dbe76 f637575a9b696f50a3303d0b9e0e5ced7c7dbe76 Merge pull request #1153 from nkukard/docs-ip-bgp-master
doc: updates for master, cherry-picked from my 3.0 branch

Tests

New test failures 1
Status Test View job Duration
Collapse Failed test_ldp_topo1 test_linux_mpls_routes History
TopoTest 2.0 Branch < 1 sec
AssertionError: Linux Kernel MPLS route output for router r1:   --- actual Linux Kernel MPLS route   +++ expected Linux Kernel MPLS route   @@ -1,5 +1,5 @@   -xx as to xx via inet 10.0.1.2 dev r1-eth0  proto xx   -xx as to xx via inet 10.0.1.2 dev r1-eth0  proto xx   -xx via inet 10.0.1.2 dev r1-eth0  proto xx   -xx via inet 10.0.1.2 dev r1-eth0  proto xx   -xx via inet 10.0.1.2 dev r1-eth0  proto xx   +xx as to xx via inet 10.0.1.2 dev r1-eth0  proto zebra   +xx as to xx via inet 10.0.1.2 dev r1-eth0  proto zebra   +xx via inet 10.0.1.2 dev r1-eth0  proto zebra   +xx via inet 10.0.1.2 dev r1-eth0  proto zebra   +xx via inet 10.0.1.2 dev r1-eth0  proto zebra assert 1 == 0
E   AssertionError: Linux Kernel MPLS route output for router r1:
      --- actual Linux Kernel MPLS route
      +++ expected Linux Kernel MPLS route
      @@ -1,5 +1,5 @@
      -xx as to xx via inet 10.0.1.2 dev r1-eth0  proto xx
      -xx as to xx via inet 10.0.1.2 dev r1-eth0  proto xx
      -xx via inet 10.0.1.2 dev r1-eth0  proto xx
(8 more lines...)

Shared artifacts

Artifact File size
SourceCode 3 MB
VersionInfo 698 bytes
topotests 238 KB