[quagga-dev 16200] Re: Proposed 8 testing Update

2016-10-06 Thread Lou Berger
On October 6, 2016 11:47:59 AM Paul Jakma wrote: On Thu, 6 Oct 2016, Lou Berger wrote: One outstanding issue for the release is operation of BGP without zebra. (The NHT change basically introduces a requirement to run BGP with zebra, while there are route server/reflector

[quagga-dev 16199] Re: Proposed 8 testing Update

2016-10-06 Thread Paul Jakma
On Thu, 6 Oct 2016, Lou Berger wrote: One outstanding issue for the release is operation of BGP without zebra. (The NHT change basically introduces a requirement to run BGP with zebra, while there are route server/reflector configs which are viable now running only BGP.) I suggest changing

[quagga-dev 16198] Re: Proposed 8 testing Update

2016-10-06 Thread Vincent JARDIN
Le 06/10/2016 à 17:01, Philippe Guibert a écrit : No. Please merge to master ASAP. > +1 +1 ___ Quagga-dev mailing list Quagga-dev@lists.quagga.net https://lists.quagga.net/mailman/listinfo/quagga-dev

[quagga-dev 16195] [PATCH] bgp: ignore NHT when bgpd has never connected zebra

2016-10-06 Thread Lou Berger
--- quagga/bgpd/bgp_nht.c | 3 ++- quagga/bgpd/bgp_zebra.c | 11 +++ quagga/bgpd/bgp_zebra.h | 2 ++ 3 files changed, 15 insertions(+), 1 deletion(-) diff --git a/quagga/bgpd/bgp_nht.c b/quagga/bgpd/bgp_nht.c index 591fbf9..c4acb66 100644 --- a/quagga/bgpd/bgp_nht.c +++

[quagga-dev 16193] Re: Proposed 8 testing Update

2016-10-06 Thread Martin Winter
I think something to address before a release, but not sure if we should hold off merging to master because of it. Lou, are you proposing to fix this before merging to master? - Martin On 6 Oct 2016, at 6:30, Lou Berger wrote: > Paul, > > On October 6, 2016 6:05:14 AM Paul Jakma

[quagga-dev 16191] Re: Proposed 8 testing Update

2016-10-06 Thread Lou Berger
Paul, On October 6, 2016 6:05:14 AM Paul Jakma wrote: ... > > One change from the last release in BGP is ANVL-BGPPLUS-17.1 related to > address used for nexthop - NHT related? Is it serious? > > Looks acceptable generally. > One outstanding issue for the release is operation of

[quagga-dev 16192] Re: Proposed 8 testing Update

2016-10-06 Thread Martin Winter
There are a few OSPFv3 issues as well ANVL-OSPFV3-8.5 & 19.11-19.16 I have not looked yet into the failure case (sorry busy this week). I would vote to address them after the merge to master. - Martin On 6 Oct 2016, at 3:02, Paul Jakma wrote: On Wed, 5 Oct 2016, Martin Winter wrote: Paul,

[quagga-dev 16190] Re: Proposed 8 testing Update

2016-10-06 Thread Paul Jakma
On Wed, 5 Oct 2016, Martin Winter wrote: Paul, Results are updated. See https://drive.google.com/drive/folders/0B8W_T0dxQfwxZFdJd1lPOXo3bHM?usp=sharing OSPFv2 issues are fixed. I haven’t looked at all the other protocols, but overall they seem to be mostly ok. Yeah, mostly unchanged.

[quagga-dev 16189] Re: Proposed 8 testing Update

2016-10-06 Thread Paul Jakma
On Wed, 5 Oct 2016, Martin Winter wrote: OSPFv2 issues are fixed. I haven’t looked at all the other protocols, but overall they seem to be mostly ok. \o/ Let me know if there are any concerns regarding a specific issue and I’ll look into it, but might both have much time this week anymore

[quagga-dev 16188] CI Testresult: PASSED (Re: [quagga-dev, 16187, v3, 3/3] tests: enhance bgpd testing with label value)

2016-10-06 Thread cisystem
Continous Integration Result: SUCCESSFUL Congratulations, this patch passed basic tests Tested-by: NetDEF CI System This is an EXPERIMENTAL automated CI system. For questions and feedback, feel free to email Martin Winter . Patches applied :

[quagga-dev 16186] [PATCH v3 2/3] bgpd: multiple label stack support

2016-10-06 Thread Philippe Guibert
From: David Lamparter Support for multiple label support handling in NLRI messages. Ability to receive and sent BGP updates with NLRI containing multiple labels. The commit follows label encoding rules, contained in RFC3107. Signed-off-by: David Lamparter

[quagga-dev 16185] [PATCH v3 1/3] lib: 3byte stream foo

2016-10-06 Thread Philippe Guibert
From: David Lamparter API enhancement to be able to write and read 3 byte series in stream structures. Signed-off-by: David Lamparter Signed-off-by: Philippe Guibert --- lib/stream.c | 77

[quagga-dev 16187] [PATCH v3 3/3] tests: enhance bgpd testing with label value

2016-10-06 Thread Philippe Guibert
bgp NLRI has a specific format for label, terminating with bottom of stack bit. This value is updated in all BGP attribute tests. A specific test has been added to check for a list of labels contained in a single NLRI. Signed-off-by: Philippe Guibert ---

[quagga-dev 16183] Re: Proposed 8 testing Update

2016-10-06 Thread Martin Winter
Paul, Results are updated. See https://drive.google.com/drive/folders/0B8W_T0dxQfwxZFdJd1lPOXo3bHM?usp=sharing OSPFv2 issues are fixed. I haven’t looked at all the other protocols, but overall they seem to be mostly ok. Let me know if there are any concerns regarding a specific issue and