Build #4

Plan branch:
Topotest PR7917 Retest

Build: #4 failed Rebuilt by Martin Winter

Stages & jobs

  1. Get Sourcecode

  2. Building Stage

  3. Basic Tests

  4. Final

    Final

Build result summary

Details

Completed
Duration
96 minutes
Labels
version=frr-7_4-dev-4514-gf40594ca7git=https_//github_com/frrouting/frr_gitbuildurl=https_//ci1_netdef_org/browse/testing-topo7917b-4branch=master
Revision
061166ac0cefd6ef79679da9f41c09d9f13efe31 061166ac0cefd6ef79679da9f41c09d9f13efe31
Total tests
7266
Failing since
#3 (Manual run by Martin Winter – )
Number of retries
1

Tests

Responsible

This build has been failing since #3
No one has taken responsibility

Code commits

Author Commit Message Commit date
Philippe Guibert Philippe Guibert 061166ac0cefd6ef79679da9f41c09d9f13efe31 061166ac0cefd6ef79679da9f41c09d9f13efe31 Merge pull request #8048 from idryzhov/fix-vrf-command
vrf: mark vrf as configured when entering vrf node
Donald Sharp Donald Sharp ed985dfd9d95cd4c0abf49dbd6a023983302d634 ed985dfd9d95cd4c0abf49dbd6a023983302d634 Merge pull request #8045 from ton31337/feature/doc_ebgp_multihop_with_nht_resolve
doc: We have to enable nht tracking via default route multihop eBGP
Igor Ryzhov <iryzhov@nfware.com> Igor Ryzhov <iryzhov@nfware.com> 18e196f6fad9dac2a6eaa3e5ab3ac3a9ec4fcceb 18e196f6fad9dac2a6eaa3e5ab3ac3a9ec4fcceb vrf: use wrappers to change VRF_CONFIGURED flag
Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
Igor Ryzhov <iryzhov@nfware.com> Igor Ryzhov <iryzhov@nfware.com> 966806294b168fdc7bbd46797c3f0a66b939f00e 966806294b168fdc7bbd46797c3f0a66b939f00e vrf: mark vrf as configured when entering vrf node
The VRF must be marked as configured when user enters "vrf NAME" command.

Otherwise, the following problem occurs:

`ip link add red type vrf table 1`

  VRF structure is allocated.

`vtysh -c "conf t" -c "vrf red"`

  `lib_vrf_create` is called, and pointer to the VRF structure is stored
  to the nb_config_entry.

`ip link del red`

  VRF structure is freed (because it is not marked as configured), but
  the pointer is still stored in the nb_config_entry.

`vtysh -c "conf t" -c "no vrf red"`

  Nothing happens, because VRF structure doesn't exist. It means that
  `lib_vrf_destroy` is not called, and nb_config_entry still exists in
  the running config with incorrect pointer.

`ip link add red type vrf table 1`

  New VRF structure is allocated.

`vtysh -c "conf t" -c "vrf red"`

  `lib_vrf_create` is NOT called, because the nb_config_entry for that
  VRF name still exists in the running config.

After that all NB commands for this VRF will use incorrect pointer to
the freed VRF structure.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
Russ White <russ@riw.us> Russ White <russ@riw.us> d887c7bf04f08b37ec122587f11cd2079cb22bd7 d887c7bf04f08b37ec122587f11cd2079cb22bd7 Merge pull request #7973 from sworleys/Pbr-More-Fixes
zebra,pbrd,doc: PBR more fixes

10 more changes...

Configuration changes

Plan configuration has changed since the last successful build. See the plan audit log for more details.

Tests

Existing test failures 1
Status Test Failing since View job Duration 
AssertionError: R2 could not peer 1.1.1.1 state expected Connect got Established  assert False == True   -False   +True
E   AssertionError: R2 could not peer 1.1.1.1 state expected Connect got Established 
    assert False == True
      -False
      +True