| Age | Commit message (Collapse) | Author |
|---|
|
|
|
ldpd: use a timer instead of sleeping in LM init
|
|
bgpd: Show PfxSnt for `show bgp <afi> <safi>` command
|
|
|
|
bgpd: When static default route is present in RIB and advertised to p…
|
|
Stop sleeping if synchronous label-manager zapi session
has trouble during init: retry using a timer instead. Move
initial label-block request to a point where the LM zapi
session is known to be running. Remove the use of the
daemon 'instance' - we're using the session_id to distinguish
the LM zapi session.
Signed-off-by: Mark Stapp <mjs@voltanet.io>
|
|
lib, zebra: add missing extern "C" {} blocks to new header files
|
|
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
|
|
JSON output was already in, just printing in CLI as well.
```
root@exit1-debian-9:~# vtysh -c 'show bgp ipv4 summary'
IPv4 Unicast Summary:
BGP router identifier 10.10.10.1, local AS number 65001 vrf-id 0
BGP table version 100003
RIB entries 200005, using 37 MiB of memory
Peers 1, using 21 KiB of memory
... MsgRcvd MsgSent TblVer InQ OutQ Up/Down State/PfxRcd PfxSnt
... 64 157 0 0 0 00:01:18 0 100003
Total number of neighbors 1
```
JSON:
```
root@exit1-debian-9:~# vtysh -c 'show bgp ipv4 summary json' \ |
> jq '.ipv4Unicast.peers."192.168.0.2".pfxSnt'
100003
```
Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
|
|
pimd: Modified rp-info json o/p
|
|
yang: Convert to `FRRouting` for organization
|
|
Signed-off-by: Renato Westphal <renato@opensourcerouting.org>
|
|
Issue:
Configuring default-originate when static default route is previously
advertised results in withdrawal of the route.
Fix :
Delete the adj-out entry for the previously advertised static
default route without sending explicit withdraw message.
Signed-off-by: kssoman <somanks@gmail.com>
|
|
bgpd: handle fs nlri over 240 bytes
|
|
doc: remove duplicated command documentation
|
|
We should be using `FRRouting` as our organization.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|
yang: Defining yang files for pim
|
|
Signed-off-by: Sarita Patra <saritap@vmware.com>
|
|
Vrf stoppage
|
|
Fix: Added a new field "rpAddress" in "show ip pim rp-info json"
Before:
"40.0.0.2":[
{
"outboundInterface":"ens224",
"group":"224.0.0.0\/4",
"source":"Static"
}
After:
"40.0.0.2":[
{
"rpAddress":"40.0.0.2",
"outboundInterface":"ens224",
"group":"224.0.0.0\/4",
"source":"Static"
}
Signed-off-by: Sarita Patra <saritap@vmware.com>
|
|
Currently the linux kernel allows you to specify the same
table id -> multiple vrf's. While I am arguing with
the kernel people about proper behavior here let's
just remove this as a possiblity from happening and
mark it a zebra stopable misconfiguration.
(Effectively we are preventing a crash down the line
as that all over FRR we assume it's a unique
mapping not a many to one).
Why fail hard? Because we hope to get the person
who misconfigured it to actually notice immediately
not hours or days down the line when shit hits the fan.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|
The function rt_netlink.c is using to lookup the vrf by
passed in table id.
I'm also going to pretend that this function is not
so awful to run when we have a large number of routes
incoming.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|
Use the old description as it looks more complete than the current.
Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
|
|
Fixes a documentation warning during compilation.
Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>
|
|
Coverity likes coverity
|
|
It's possible(but unlikely) that a read of data from the
network will give us bogus data. Don't automatically
just trust the data size from the network and limit
the read to the size of the buffer we have in play.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|
There are a couple of switch statements in netlink_route_info_encode
in zebra_fpm_netlink.c that had logically dead code. We have
a switch statement let's take actual advantage of it instead
of doing gyrations to what we want.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
|
|
lib: make atomlist code C++ compatible
|
|
the nlri flowspec above 240 bytes size was not handled.
Over 240 bytes, the length is 2 bytes length, and a calculation must be
done to obtain the real length. This commit handles it appropriately.
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
|
|
bgpd: Enforce self-next-hop check in next-hop update.
|
|
pimd: Implement watermark warning for igmp group count and add igmp group count
|
|
... by dropping seqlock.h from the header's includes; it's only needed
in the C code in frrcu.c.
Signed-off-by: David Lamparter <equinox@diac24.net>
|
|
... by using `atomic_atomptr_t`. Other ideas seemed worse.
Signed-off-by: David Lamparter <equinox@diac24.net>
|
|
ospf6d: fix SA warnings
|
|
truncation warnings on old compilers
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
zebra/fpm: fix shutdown and add more documentation
|
|
doc: workflow updates
|
|
Clean up some SA warnings in use of ospf6_lsa_unlock()
Signed-off-by: Mark Stapp <mjs@voltanet.io>
|
|
ldpd: don't drop packets coming through a broken LSP
|
|
Just writing down what seems to be established procedure.
Signed-off-by: David Lamparter <equinox@diac24.net>
|
|
Signed-off-by: David Lamparter <equinox@diac24.net>
|
|
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
- Fix 1 byte overflow when showing GR info in bgpd
- Use PATH_MAX for path buffers
- Use unsigned specifiers for uint16_t's in zebra pbr
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
Take care of some more complicated cases by hand
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
Replace sprintf with snprintf where straightforward to do so.
- sprintf's into local scope buffers of known size are replaced with the
equivalent snprintf call
- snprintf's into local scope buffers of known size that use the buffer
size expression now use sizeof(buffer)
- sprintf(buf + strlen(buf), ...) replaced with snprintf() into temp
buffer followed by strlcat
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
Coccinelle needs to know about complicated macros to understand certain
code paths, add some more macros there.
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
|
|
lib/zlog: fix coverity warning
|
|
Last patch sent in on the mailing list was in July 2018. If someone
sends a patch we can still pick it up, but practiced reality is quite
clearly PRs on github.
Signed-off-by: David Lamparter <equinox@diac24.net>
|
|
Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
|
|
bgpd: Do not discard an UPDATE if the global nexthop is set to ::
|
