[quagga-dev 16458] CI Testresult: FAILED (Re: [quagga-dev, 16455, 23/23] bgpd: set restart_time from capnproto API)

2016-12-02 Thread cisystem
Continous Integration Result: FAILED See below for issues. This is an EXPERIMENTAL automated CI system. For questions and feedback, feel free to email Martin Winter . Patches applied : Patchwork 2233: http://patchwork.quagga.net/patch/2233

[quagga-dev 16457] CI Testresult: FAILED (Re: [quagga-dev, 16421, v3, 5/5] bgpd: vpnv4 af exports extended communities)

2016-12-02 Thread cisystem
Continous Integration Result: FAILED See below for issues. This is an EXPERIMENTAL automated CI system. For questions and feedback, feel free to email Martin Winter . Patches applied : Patchwork 2217: http://patchwork.quagga.net/patch/2217

[quagga-dev 16456] [PATCH 12/23] bgpd: add autogenerated files

2016-12-02 Thread Philippe Guibert
From: David Lamparter This commit introduces .c and .h files issued from bgp.capnp scheme file. Generating .c files from this scheme file requires capnp compiler, and also some custom libraries that, at the time of this writing are yet under development. So, practically, in

[quagga-dev 16450] [PATCH 17/23] bgpd: support for max_mpath vrf field configuration

2016-12-02 Thread Philippe Guibert
This commit enhances the ability to configure from capnproto the maximum path field per vrf. The ability to configure multipath for an afi/safi is also made possible. Signed-off-by: Philippe Guibert --- bgpd/bgp.bcapnp.c | 61

[quagga-dev 16443] [PATCH 07/23] lib: add basic capn config access protocol

2016-12-02 Thread Philippe Guibert
From: David Lamparter This commit introduces the qzc.capnp schema file. This schema file is only here for information, despite it is possible to automate c files generation, from this scheme file. In addition, qzc framework needs some server API to handle qzc

[quagga-dev 16454] [PATCH 22/23] bgpd: add a notification message on received bgp notifications

2016-12-02 Thread Philippe Guibert
An handler is added to signify that a remote notification has been received, meaning that local BGP speaker should be shutdown. Signed-off-by: Philippe Guibert --- bgpd/bgp_debug.c | 11 +++ bgpd/bgp_zmq.c | 9 + bgpd/bgpd.h | 12 +++-

[quagga-dev 16453] [PATCH 21/23] bgpd: add capnproto support for bgp update-source

2016-12-02 Thread Philippe Guibert
This commit handles the case where an incoming updateSource IP address is received for setting as bgp update-source IP address. Signed-off-by: Philippe Guibert --- bgpd/bgp.bcapnp.c | 54 +++--- 1 file changed, 51

[quagga-dev 16452] [PATCH 20/23] bgpd: add vty header under bgpd.h header file

2016-12-02 Thread Philippe Guibert
Because some warnings tell that vty structure is not known, it is advised to self-content the header so that vty structure is known. Smooth alternative would have been to declare structure name on top of bgpd.h. Signed-off-by: Philippe Guibert --- bgpd/bgpd.h | 1 +

[quagga-dev 16455] [PATCH 23/23] bgpd: set restart_time from capnproto API

2016-12-02 Thread Philippe Guibert
The graceful restart time is configurable by using capnproto API Signed-off-by: Philippe Guibert --- bgpd/bgp.bcapnp.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/bgpd/bgp.bcapnp.c b/bgpd/bgp.bcapnp.c index 1667033b823f..acdc473a2f7c 100644

[quagga-dev 16438] [PATCH 04/23] *: force the linker to not check for unresolved symbols at link

2016-12-02 Thread Philippe Guibert
This commit bypasses the link check in case a shared library is to be used in conjunction with one of the quagga daemons. This is the case of zero ma library that may or not be available on some systems. Adding -Wl,-unresolved-symbols=ignore-in-shared-libs to the linker flags permits compiling

[quagga-dev 16446] [PATCH 10/23] bgpd: enable use of capn/zmq interface

2016-12-02 Thread Philippe Guibert
From: David Lamparter This commit enables the usage of capn/zmq API on bgp daemon, by linking and using capn/zmq APIs, through using specific BGP configuration parameters. It will be possible to receive queries by using a zmq socket mentioned by -Z parameter. Also,

[quagga-dev 16451] [PATCH 19/23] bgpd: declare header where FILTER_MAX is defined

2016-12-02 Thread Philippe Guibert
Add include header filter.h file, because bgpd.h defines structures with FILTER_MAX as define. Signed-off-by: Philippe Guibert --- bgpd/bgpd.h | 1 + 1 file changed, 1 insertion(+) diff --git a/bgpd/bgpd.h b/bgpd/bgpd.h index ad6a6652784e..ba870a025ffb 100644 ---

[quagga-dev 16448] [PATCH 16/23] bgpd: enable iteration of routes in VRF

2016-12-02 Thread Philippe Guibert
From: David Lamparter This enhancement to the bgp.capnp scheme files permits bgp daemon to handle get requests to parse a routing table. It is possible to retrieve the next element of a table, by reusing the passed as parameter element, then, the query will

[quagga-dev 16441] [PATCH 06/23] lib: capnp files are copied

2016-12-02 Thread Philippe Guibert
This patch integrates c files derived from qzc.capnp scheme file. This C version is provided an needs to be syncronised with the associated qzc.capnp file. Signed-off-by: Philippe Guibert --- lib/qzc.capnp.c | 466

[quagga-dev 16444] [PATCH 13/23] bgpd: sanity checks around capnproto exchange

2016-12-02 Thread Philippe Guibert
This commit contains some modifications on supposed to be generated files, as the default files may contain some non initialised variables, it should lead to erratic behaviour on some circumstances. Signed-off-by: Philippe Guibert --- bgpd/bgp.bcapnp.c | 20

[quagga-dev 16442] [PATCH 11/23] bgpd: add BGP configuration file with and without debugging

2016-12-02 Thread Philippe Guibert
This commit brings default bgp config files that can be used, in order to get default debugging setup, or not. There is no configuration inside. Signed-off-by: Philippe Guibert --- bgpd/Makefile.am | 2 +- bgpd/bgpd.conf.sample3 | 7 +++

[quagga-dev 16436] [PATCH 03/23] tests: ZeroMQ code

2016-12-02 Thread Philippe Guibert
From: David Lamparter The test consists in sending a message from a req(uest) to a rep(ly). The rep will dump the received message and send back an ACK to say it has well received the message. To perform the zeromq test on your system, please do the following.

[quagga-dev 16439] [PATCH 08/23] lib: qzc debug log are conditioned with a debug variable

2016-12-02 Thread Philippe Guibert
This variable can be set by compilation, if there is a need to monitor qzc exchanges. Signed-off-by: Philippe Guibert --- lib/qzc.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/qzc.c b/lib/qzc.c index 358b7e419eef..7425743f8cea 100644

[quagga-dev 16437] [PATCH 05/23] tests: update makefile to link with zmq files

2016-12-02 Thread Philippe Guibert
The introduction of ZMQ library has led to automatic linking with all other daemons and test programs. This modification is taking into account the link with test programs. Signed-off-by: Philippe Guibert --- tests/Makefile.am | 42

[quagga-dev 16433] [PATCH 00/23] Quagga CapnProto API

2016-12-02 Thread Philippe Guibert
Hi all, This set of patches contains adaptations to BGP daemon, to bring a new API of management, based on capnproto protocol. The API implementation relies on capnproto protocol for data serialisation, since it is able to receive and send simple and complex data structures. The emission

[quagga-dev 16440] [PATCH 09/23] bgpd: add capnproto information on doc file related to implementation

2016-12-02 Thread Philippe Guibert
Some information is appended to bgp implementation to explain about newly added files related to capnproto. Signed-off-by: Philippe Guibert --- bgpd/IMPLEMENTATION.txt | 18 ++ 1 file changed, 18 insertions(+) diff --git a/bgpd/IMPLEMENTATION.txt

[quagga-dev 16434] [PATCH 01/23] configure: add with-zeromq and with-ccapnproto configuration parameter

2016-12-02 Thread Philippe Guibert
Those config parameters permit usage of zeromq library and capnproto library. Those two config parameters are necessary in order to use capnproto configuration interface that will be needed by BGP. Signed-off-by: David Lamparter Signed-off-by: Philippe Guibert

[quagga-dev 16431] [PATCH 07/23] lib: add basic capn config access protocol

2016-12-02 Thread Philippe Guibert
From: David Lamparter This commit introduces the qzc.capnp schema file. This schema file is only here for information, despite it is possible to automate c files generation, from this scheme file. In addition, qzc framework needs some server API to handle qzc

[quagga-dev 16432] [PATCH 10/23] bgpd: enable use of capn/zmq interface

2016-12-02 Thread Philippe Guibert
From: David Lamparter This commit enables the usage of capn/zmq API on bgp daemon, by linking and using capn/zmq APIs, through using specific BGP configuration parameters. It will be possible to receive queries by using a zmq socket mentioned by -Z parameter. Also,

[quagga-dev 16430] [PATCH 06/23] lib: capnp files are copied

2016-12-02 Thread Philippe Guibert
This patch integrates c files derived from qzc.capnp scheme file. This C version is provided an needs to be syncronised with the associated qzc.capnp file. Signed-off-by: Philippe Guibert --- lib/qzc.capnp.c | 466

[quagga-dev 16425] [PATCH 03/23] tests: ZeroMQ code

2016-12-02 Thread Philippe Guibert
From: David Lamparter The test consists in sending a message from a req(uest) to a rep(ly). The rep will dump the received message and send back an ACK to say it has well received the message. To perform the zeromq test on your system, please do the following.

[quagga-dev 16423] [PATCH 01/23] configure: add with-zeromq and with-ccapnproto configuration parameter

2016-12-02 Thread Philippe Guibert
Those config parameters permit usage of zeromq library and capnproto library. Those two config parameters are necessary in order to use capnproto configuration interface that will be needed by BGP. Signed-off-by: David Lamparter Signed-off-by: Philippe Guibert

[quagga-dev 16422] [PATCH 00/23] Quagga CapnProto API

2016-12-02 Thread Philippe Guibert
Hi all, This set of patches contains adaptations to BGP daemon, to bring a new API of management, based on capnproto protocol. The API implementation relies on capnproto protocol for data serialisation, since it is able to receive and send simple and complex data structures. The emission

[quagga-dev 16426] [PATCH 05/23] tests: update makefile to link with zmq files

2016-12-02 Thread Philippe Guibert
The introduction of ZMQ library has led to automatic linking with all other daemons and test programs. This modification is taking into account the link with test programs. Signed-off-by: Philippe Guibert --- tests/Makefile.am | 42

[quagga-dev 16429] [PATCH 09/23] bgpd: add capnproto information on doc file related to implementation

2016-12-02 Thread Philippe Guibert
Some information is appended to bgp implementation to explain about newly added files related to capnproto. Signed-off-by: Philippe Guibert --- bgpd/IMPLEMENTATION.txt | 18 ++ 1 file changed, 18 insertions(+) diff --git a/bgpd/IMPLEMENTATION.txt

[quagga-dev 16428] [PATCH 08/23] lib: qzc debug log are conditioned with a debug variable

2016-12-02 Thread Philippe Guibert
This variable can be set by compilation, if there is a need to monitor qzc exchanges. Signed-off-by: Philippe Guibert --- lib/qzc.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/qzc.c b/lib/qzc.c index 358b7e419eef..7425743f8cea 100644

[quagga-dev 16427] [PATCH 04/23] *: force the linker to not check for unresolved symbols at link

2016-12-02 Thread Philippe Guibert
This commit bypasses the link check in case a shared library is to be used in conjunction with one of the quagga daemons. This is the case of zero ma library that may or not be available on some systems. Adding -Wl,-unresolved-symbols=ignore-in-shared-libs to the linker flags permits compiling

[quagga-dev 16424] [PATCH 02/23] lib: ZeroMQ read event

2016-12-02 Thread Philippe Guibert
From: David Lamparter This commit introduces 2 new files, and introduces the compilation flags for compiling with zeromq and capnproto dependencies. Signed-off-by: David Lamparter Signed-off-by: Philippe Guibert

[quagga-dev 16419] [PATCH v3 3/5] bgpd: VRF vty configuration, RIB table creation

2016-12-02 Thread Philippe Guibert
This commit introduces the BGP VRF configuration, and BGP VRF RIB table. It includes the ability for a BGP to configure its own route distinguisher ( aka VRF). New vty commands introduced: (config-router)# vrf rd This structure permits configuring import and export route targets, which is

[quagga-dev 16417] [PATCH v3 1/5] lib: declare BGP_VRF_NODE as vty subnode

2016-12-02 Thread Philippe Guibert
This new node is to be put under router bgp subnode. On top of this, usual exit and end commands are also taking into account bgp vrf subnode. Signed-off-by: Philippe Guibert --- lib/command.c | 3 +++ lib/command.h | 1 + lib/vty.c | 1 + 3 files changed, 5

[quagga-dev 16418] [PATCH v3 2/5] vtysh: vty adaptations for new bgp vrf node

2016-12-02 Thread Philippe Guibert
This commit includes changes related to introduce new subnode under bgp called vrf <>. This node permits configuring route distinguishers and route target under the subnode, by using vtysh command tool. Signed-off-by: Philippe Guibert --- vtysh/extract.pl.in | 2 ++

[quagga-dev 16420] [PATCH v3 4/5] bgpd: vty adaptations regarding rd and rt configuration

2016-12-02 Thread Philippe Guibert
A new subnode under router bgp command has been created: vrf <>. This enhancement permits creating a VRF instance specific to VRF. Then it permits setting specific configuration to vrf entity. This is the case for route distinguisher, but also route target. vrf rd no rd rt import rt export

[quagga-dev 16421] [PATCH v3 5/5] bgpd: vpnv4 af exports extended communities

2016-12-02 Thread Philippe Guibert
If the user preconfigures a vrf with import and export rules, before creating a new network on vpnv4 address, then BGP update emitted will contain the exported rts mentioned by the vrf export rule. Signed-off-by: Philippe Guibert --- bgpd/bgp_route.c | 11 +++

[quagga-dev 16416] [PATCH v3 0/5] BGP VRF Route Distinguisher configuration

2016-12-02 Thread Philippe Guibert
This series of patches introduces the ability for a BGP router, to configure a route distinguisher and associate import and export route targets. This permits for a BGP update to be sent with BGP extended communities associated to the exported route targets of the RD. This permits for a BGP update