Minor adjustments to address CI warnings

fabricd: Disabling OpenFabric optimization to avoid considering T0 devices only as DNRs

Signed-off-by: Rodny Molina <rmolina@linkedin.com>

isisd: Provide statistics over number of generated purges

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Provide statistics over number of LSP regenerations

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Provide statistics about retransmitted LSPs

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Provide statistics about sent/received PDU count

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

fabricd: Update neighbor list on adjacency state change

We should update our neighbors list immediately when an adjacency
changes state, not when we run SPF.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

fabricd: make triggered csnp delay configurable

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

fabricd: Improve LSP flooding log

Also track when we received an LSP as do not reflood, as well as the
time when we last considered flooding it.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

fabricd: Add `show openfabric flooding` command

Add a command to show to what neighbors an LSP has been flooded.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Make lspid_print non-static

`lspid_print` is useful in other places, so make it available
in `isis_lsp.h`.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Move code to find LSP out of show database

Finding an LSP by its id is useful not only for the
`show isis database` command.

So move it out into its own function to make it reusable.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

fabricd: Populate NL from adjdb, not spf

We should really populate the neighbor list for the flooding
optimization from our local adjacency database and not from
a one-hop SPF.

If we use SPF, we may end up never exchanging information with
some neighbors since the bidirectional connection check for spf
fails, since LSPs did not get exchanged.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Provide better flooding debugging

Show from where LSP flooding is triggered.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Remove unused t_send_lsp timer

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Add debug output for tx-queues

To allow easier debugging of LSP transmission scheduling, add a debug
mode where all tx-queue insertions/deletions are logged.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Merge pull request #3438 from opensourcerouting/bgp-rfapi-default-value

bgpd: don't show default value in configuration

Merge pull request #3437 from opensourcerouting/bugfix/topotests-use-copy

tests/topotests: Use copied tests in Docker

Merge pull request #3436 from opensourcerouting/bugfix/topotest-sbin-dir

tests/topotests: Also search sbin for modprobe

Merge pull request #3421 from pguibert6WIND/bgp_lp_finish_called_too_early

bgpd: delay destruction of label pool

Merge pull request #3425 from chiragshah6/evpn_dev1

bgpd: set attribute change flag to evpn imported route

bgpd: don't show default value in configuration

Don't show the configuration line `rfp full-table-download off` by
default as it is not the default value, instead only show
`rfp full-table-download on` (the non-default value) when it is
configured.

This standardizes this knob to the FRR default behavior.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

tests/topotests: Also search sbin for modprobe

We call `modprobe -n` to check if mpls modules are available to be
loaded. We do this as normal user, to only ask for root permissions
if we are actually loading the module.

This breaks if `modprobe` is in `/sbin` and normal users don't have
sbin in path.

So add `/sbin` to the search path to work around this.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

tests/topotests: Use copied tests in Docker

If we mount the tests into the container from the host, we also
mount any `*.pyc` files with them, which will lead to issues
as the mount is done read-only to avoid any changes to the host.

Since the tests are now integrated and we already create a writeable
copy of the FRR tree, just use the tests from the FRR tree to avoid
this issue.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Merge pull request #3419 from opensourcerouting/freebsd-route-warning-fix

zebra: fix some boot up warnings on FreeBSD

bgpd: set attribute change flag to evpn imported

EVPN route's attribute changes,
mark attribute change flag to imported unicast route.

A scenario where AS_PATH attribute have changed for an EVPN type-5
route, set attribute change
to imported route.

Ticket:CM-23008
Reviewed By:
Testing Done:
Validated via marking EVPN route with AS_PATH prepand.
At the receiving VTEP, ensure attribute change flag is set to
imported unicast route and bgp update sent to VTEPs subsequent
bgp peers with AS_PATH prepend update.

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

zebra: don't log errors on unsupported medias

When using `SIOCGIFMEDIA` check for `EINVAL`, otherwise we might print
an error message on an unsupported interface.

FreeBSD source code reference:
https://github.com/freebsd/freebsd/blob/master/sys/net/if_media.c#L300

And:
https://github.com/freebsd/freebsd/blob/8cb4b0c0181bd45318ee8977f77aea90c53bb224/usr.sbin/rtsold/if.c#L211

  /*
   * EINVAL simply means that the interface does not support
   * the SIOCGIFMEDIA ioctl. We regard it alive.
   */

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bgpd: delay destruction of label pool

label pool finalisation must be delayed after route deletion on bgp.
otherwise a crash will happen, while labels will be released.

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

Merge pull request #3418 from pguibert6WIND/bgp_rfapi_path_info_extra_free

bgpd: reuse bgp_path_info_extra_free() routing in rfapi

zebra: refactor route socket message handling

Some address types were not being skipped triggering a warning log
message, so lets refactor this code to properly handle known and unknown
types.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bgpd: reuse bgp_path_info_extra_free() routing in rfapi

rfapi code should use bgp_path_info_extra_free() routine.

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

zebra: always define ROUNDUP and ROUND_TYPE

Move the declaration of ROUNDUP and ROUND_TYPE to outside of
`ifdef SA_SIZE`. We'll use these definitions in the next commit.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

Merge pull request #3372 from nitinsoniism/show_evpn_mac_vni_all_detail

zebra: Add "show evpn mac vni all detail" command

Merge pull request #3174 from opensourcerouting/feature/isis-triggered-hello

Feature: IS-IS triggered hello

Merge pull request #3410 from opensourcerouting/bugfix/topotest-docker-build-context

Revert "tests/topotests: Change docker build context"

Merge pull request #3340 from nitinsoniism/show_evpn_vni_detail

zebra: Add "show evpn vni detail" command

topotests: Update json_cmp_result to return readable result

Before this update, json_cmp_results which were formatted as strings
would not show the error mesage but just an object reference.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

topotests/isis-topo1: Set useful IS-IS debugs

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: trigger hellos on adjacency state change

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: add send_hello_sched function

Add a function send_hello_sched so that the logic for scheduling a
hello is not replicated inconsistently into different locations.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Merge pull request #3413 from chiragshah6/evpn_dev1

zebra: dup addr detect clear cmd non-zero return

Merge pull request #3357 from ton31337/fix/optional_args_for_extcommunity-lists

bgpd: Remove extcommunity-list by name without typing full rule

lib: add thread_timer_remain_msec function

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Move dr election timer to the end of isis_dr_commence

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: reduce code duplication for levels

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Add level&circuit thread argument to circuit

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

zebra: dup addr detect clear cmd non-zero return

Clear dup address vni needs to return non-zero value
in case of command is not successful.

Ticket:CM-23122
Testing Done:
run clear command and check upon failure return code is non-zero.

root@TORS1:~# vtysh -c "clear evpn dup-addr vni 1000 ip 45.0.1.26"
% Requested IP's associated MAC 00:01:02:03:04:05 is still in duplicate
% state
root@TORS1:~# echo $?
1

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

Merge pull request #3370 from pguibert6WIND/default_vrf_initialization

Default vrf initialization

Merge pull request #3404 from donaldsharp/nexthop_cleanup

lib: Cleanup nexthop2str code to be consistent

Merge pull request #3402 from pacovn/Coverity_1475198_1475199

pbrd sharpd: return check fixes (Coverity 1475198 1475199)

pbrd: return check (Coverity 1475199)

Signed-off-by: F. Aragon <paco@voltanet.io>

sharpd: return check (Coverity 1475198)

Signed-off-by: F. Aragon <paco@voltanet.io>

Revert "tests/topotests: Change docker build context"

This reverts commit 659782730bffea2d21e2fa22550db3166017061e.

Apparently, the build context is inferred from the Dockerfile path.
Yay for sensible documentation. :/

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Merge pull request #3409 from opensourcerouting/feature/cleanup-topotest-docker-docs

Feature/cleanup topotest docker docs

Merge pull request #3408 from opensourcerouting/revert-empty-init

Revert "isisd lib ospfd pbrd python: fix empty init"

tests/topotests: Set -o cache_dir=/tmp

We are mounting the topotests from the host into the container
and mark them as read-only to avoid any modifications to the host.

Pytest tries to create a .pytest_chache directory in that location
which fails because of the read-only mount.

Configure a different cache dir which is writable to resolve this.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

tests/topotests: Change docker build context

In order to support automated builds, the build context needs to be
the repository root.

So adapt our dockerfile and buildscript for this. Also, add files which
should not be included into the build context in .dockerignore to limit
the size of the build context to something sensible.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Revert "isisd lib ospfd pbrd python: fix empty init"

This reverts commit 48944eb65e1d1ced03d46121d923d9d613a480d5.

We're using GNU C, not ISO C - and this commit triggers new (real)
warnings about {0} instead of bogus ones about {}.

Signed-off-by: David Lamparter <equinox@diac24.net>

Merge pull request #3405 from LabNConsulting/working/master/fix-vrf

bgpd: partially revert e23b9ef6d271223d29c7f91a10d98aa6dcd252b3

tests/topotests: Update documentation

The documentation needs an update to match the integrated version.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Merge pull request #3378 from opensourcerouting/remove-config-lock

*: remove the configuration lock from all daemons

Merge pull request #3386 from pguibert6WIND/bgp_fs_pbr_ignores_null_ip

bgpd: bgp pbr ignores bgp fs entries with 0.0.0.0 ips written.

Merge pull request #3390 from pguibert6WIND/bgp_flowspec_new_redirect_ip_draft

bgpd: add new draft for redirect ip for flowspec

Merge pull request #3403 from opensourcerouting/bfd-fix-mhop-bsd

bfdd: fix multihop on FreeBSD

lib: Cleanup nexthop2str code to be consistent

We sometimes store ifindex information in the NEXTHOP_TYPE_IPV[4|6]
so let's let us display that information as well when dumping
a nexthop.

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

Merge pull request #3396 from opensourcerouting/feature/topotests-docker

feature: docker image for topotests

bgpd: partially revert e23b9ef6d271223d29c7f91a10d98aa6dcd252b3
      previous change was to fix rnh module in Zebra for leaked routes
      this reverts that fix, so probably reintroduces the problem.

Signed-off-by: Lou Berger <lberger@labn.net>

bfdd: fix multihop on FreeBSD

Some systems have a extra `struct sockaddr*` field (`_len`) which must
be filled in order for the peer lookup to work.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bgpd: bgp pbr ignores bgp fs entries with 0.0.0.0 ips written.

when converting bgp fs entries to bgp pbr entries, the fields of the
flowspec are analysed. In the case src ip or dst ip is set to 0.0.0.0,
that field is ignored, thus preventing from injecting a rule that can
not be injected into the pbr. This can be done by avoiding mentioning
the field in the bitmask structure used to convert data to pbr entries.

PR=61620
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Acked-by: Emmanuel Vize <emmanuel.vize@6wind.com>

bgpd: add new draft for redirect ip for flowspec

that new option will overwrite simpson draft. There is a new ecommunity
option whose type is 0x1 and subtype is 0xc. That option is defined
here on iana.org/assignments/bgp-extended-communities page:
- bgp-extended-communities.xhtml#trans-ipv4
It contains the IP address to redirect traffic to. The understanding of
the draft is the following one:
- if that community is only present, then the ip contained inside will
be chosen as nexthop.
- if that community is provided along with simpson extended community,
then only the new redirect ip draft will be used. however, both will be
displayed.
- in other cases, if there is only the simpson extended community, then
the nexthop IP of the NLRI will be chosen.

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

zebra: handle neighbor delete from kernel for ipv4 link-local

Problem reported that kernel neighbor entries could end up in "FAILED"
state when the neighbor entry was deleted.  This fix handles the
notification of the event from netlink messages and re-inserts the
deleted entry.

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

Merge pull request #3391 from pguibert6WIND/doc_ldp_prefered_connection

doc: add ldp information over transport connection

Merge pull request #3394 from karamalla0406/frr3360

bgpd: The default IP route not advertised with configured RD

Merge pull request #3383 from qlyoung/fix-static-routes-again

staticd: fix null0 routes (again)

Merge pull request #3342 from opensourcerouting/nb-operational-data

Northbound: improved support for YANG-modeled operational data

Merge pull request #3397 from mjstapp/fix_stream_macros

libs,bgpd: remove deprecated stream lib macros

lib,bgpd: remove deprecated stream lib macros

A couple of deprecated lib/stream macros have aged out; remove
them, and replace the one remaining use.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

topotests: Only set -t when input is a terminal

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

topotests: Adapt docker changes for integrated tests

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

docker/frr-topotests: Fix missing $ in if check

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

docker/compile_frr: Don't build with disable shared

With libyang, we need to build the frr_user_types.so libyang
plugin for FRR to work.

This means we cannot build with disable shared.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

docker: Improve README and frr-topotests usage

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Dockerfile: Install libyang

Recent versions of FRR require libyang to build. So install it from
the FRR CI servers.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Docker: Fail if compilation is unsucessful

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Docker: Document additional environment variables

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Docker: Use pytest as default command

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Docker: move openvswitch setup to separate script

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Docker: Allow executing topotests while in subdir of FRR repo

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Docker: really cleanup when TOPOTEST_CLEAN is specified

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Docker: Update buildscript not to delete old images

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Docker: Call the run script frr-topotests.sh

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Dockerfile: Use iputils-ping instead of inetutils-ping, install strace

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Docker: Update buildscripts to be more efficient

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

docker: Install topotests into image

Also: Change directory layout so we can do an indempotent sync from host
later.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

docker: reindent shell scripts

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

docker: Run container with its own network namespace

For normal operation, there is no need for the container to run
in the host network environment or to have access to the users
X Session.

To accomodate usecases which need this, there is now a `TOPOTEST_OPTIONS`
environment variable to provide additional options to the docker run
command.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Dockerfile: Move to project root and combine RUNs

According to https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#run
`apt-get update` should always be combined with `apt-get install` in
the same `RUN` statement, to avoid installation of outdated packages.

Also, combine some more `RUN`s together as to avoid fewer layers.

Finally, remove some comments which are superflous. Comments shouldn't
describe what is done, especially when it's obvious. They should
explain why something is done.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

docker: add support for running inside container

Based on the original @cfra idea of running topotest on docker.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

Merge pull request #3384 from donaldsharp/topotests

Topotests

bgpd: The default IP route not advertised with configured RD

When "default-originate ipv4" is configured, a type-5 route is installed in
the local node and advertised to the peer with auto-rd.

When the above was followed by configuring an RD in IP VRF, Type-5 are
generated for only the non-default routes.

Fixed this issue by withdrawing the default route with auto-rd and advertising
 the route with confiured RD.

Signed-off-by: Kishore Aramalla karamalla@vmware.com