Testing PullReq against Topology Tests

Build: #308 was successful Custom build by Github Hook with revision master

Build result summary

Details

Completed
Duration
24 minutes
Flags
Custom revisionCustom build
Labels
version=frr-5_1-dev-1111-gee55f64dcgit=https_//github_com/frrouting/topotests_gitbuildurl=https_//ci1_netdef_org/browse/frr-topopr-cf-308branch=master
Revisions
FRR - Topotests
master master
FRR - FreeRangeRouting
ee55f64dc54bcec0c165836f47f2e2a45b7bdec8 ee55f64dc54bcec0c165836f47f2e2a45b7bdec8
Total tests
873
Successful since
#305 ()

Tests

Custom variables

Custom revision
master master
Name Value
github_base_sha f8daeea19ea0fe79fbdb7231d7a7bc34e5a4f0ac
github_pullreq 117
github_branch master
github_repo FRRouting/topotests
github_merge_sha f7e414eb6edda58c044048f6e0d4bce3cb5b2de2

Comments

  1. Github Hook

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

    Request to merge from opensourcerouting/topotests
    Merge Git Commit ID f7e414eb6edda58c044048f6e0d4bce3cb5b2de2 on top of base Git Commit ID f8daeea19ea0fe79fbdb7231d7a7bc34e5a4f0ac

Code commits

FRR - Topotests
Author Commit Message Commit date
Rafael Zalamena <rzalamena@users.noreply.github.com> Rafael Zalamena <rzalamena@users.noreply.github.com> f8daeea19ea0fe79fbdb7231d7a7bc34e5a4f0ac f8daeea19ea0fe79fbdb7231d7a7bc34e5a4f0ac Merge pull request #114 from LabNConsulting/working/direct+moreinfo
Prepare for timing changes seen in #2122
FRR - FreeRangeRouting
Author Commit Message Commit date
Russ White <russ@riw.us> Russ White <russ@riw.us> ee55f64dc54bcec0c165836f47f2e2a45b7bdec8 ee55f64dc54bcec0c165836f47f2e2a45b7bdec8 Merge pull request #2767 from donaldsharp/bgp_timers
bgpd: Intentionally stop some timers on instance removal
Donald Sharp Donald Sharp 6960cd09c6cc24a9acaa199596a895170d155912 6960cd09c6cc24a9acaa199596a895170d155912 Merge pull request #2769 from ajones-rvbd/ajones-frr-reload
tools/frr: make frr reload error message clearer on missing frr-reload.py
Arthur Jones <arthur.jones@riverbed.com> Arthur Jones <arthur.jones@riverbed.com> 8b5f787e8ea763d826e42ae007bdb48bece764ac 8b5f787e8ea763d826e42ae007bdb48bece764ac tools/frr: make frr reload error message clearer on missing frr-reload.py
Only frr-reload.py pulls in a python depenedency for frr, we can
reduce the size of the base frr package by a lot if we separate
out frr-pythontools.  When we do this, we get a somewhat cryptic
error message when frr-reload.py is missing on frr reload.

Here, we pull the error message from frr-reload script, which is
much clearer.

Testing done:

frr reload both with and without the frr-reload.py script, see
the frr-reload message when missing and it runs frr-reload.py when
not missing.

Signed-off-by: Arthur Jones <arthur.jones@riverbed.com>
Donald Sharp Donald Sharp 037f162fe9005d655c9f73cbaff37dea7ee560fa 037f162fe9005d655c9f73cbaff37dea7ee560fa Merge pull request #2766 from lyq140/branch1
ripd: null check key

Shared artifacts

Artifact File size
VersionInfo 785 bytes
SourceCode 4 MB
topotests 428 KB