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

2016-10-03 Thread Paul Jakma
Hi Martin, On Sun, 2 Oct 2016, Martin Winter wrote: I did a full test run of our RFC compliance testbed against volatile/patch-tracking/8/proposed/ff-2016093002 (Commit e4adc26) Results are at https://drive.google.com/drive/folders/0B8W_T0dxQfwxZFdJd1lPOXo3bHM?usp=sharing The biggest issues

[quagga-dev 16162] Re: [PATCH] Extend BGP_SEND_ASPATH_CHECK to cover confederations

2016-10-03 Thread Philippe Guibert
On Thu, Sep 29, 2016 at 7:25 PM, Thorvald Natvig wrote: Hello Thorvald, > Extend the check for BGP_SEND_ASPATH_CHECK to also cover confederations. .. > -#endif /* BGP_SEND_ASPATH_CHECK */ > >/* If we're a CONFED we need to loop check the CONFED ID too */ >if

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

2016-10-03 Thread Vincent JARDIN
Le 03/10/2016 à 15:08, Paul Jakma a écrit : My inclination is to release as is. Document regressions. Try fix them later. +1 ___ Quagga-dev mailing list Quagga-dev@lists.quagga.net https://lists.quagga.net/mailman/listinfo/quagga-dev

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

2016-10-03 Thread Martin Winter
On 3 Oct 2016, at 6:08, Paul Jakma wrote: Hi Martin, On Sun, 2 Oct 2016, Martin Winter wrote: I did a full test run of our RFC compliance testbed against volatile/patch-tracking/8/proposed/ff-2016093002 (Commit e4adc26) Results are at

[quagga-dev 16165] Re: [PATCH] Extend BGP_SEND_ASPATH_CHECK to cover confederations

2016-10-03 Thread Thorvald Natvig
Hi, The #define is, by default, undefined. It's designed to cover re-announcements of routes learned that contain your own AS. Unfortunately, that #define only covers half the code; it's missing for the code that deals with confederations. Let's use an example: My company has AS 123, and we

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

2016-10-03 Thread Vincent Jardin
Le 4 octobre 2016 3:58:59 AM Alexis Rosen a écrit : On Oct 3, 2016, at 11:21 AM, Vincent JARDIN wrote: Le 03/10/2016 à 15:08, Paul Jakma a écrit : My inclination is to release as is. Document regressions. Try fix them

[quagga-dev 16159] Re: CI Testresult: FAILED (Re: [quagga-dev, 16135] Extend BGP_SEND_ASPATH_CHECK to cover confederations)

2016-10-03 Thread Martin Winter
On 1 Oct 2016, at 12:28, Thorvald Natvig wrote: Hi, This is my first patch to Quagga, so I'm not quite familiar with the process. What's the next action to help this get reviewed? Should I resubmit the patch so it gets tested again? If you are able to find the issue, then yes please just

[quagga-dev 16160] Re: Doing something different wrt patch integration

2016-10-03 Thread Paul Jakma
Hi Lou, On Fri, 30 Sep 2016, Lou Berger wrote: Paul, Just restating things I've said here before : - I don't the issue is backlog It definitely was an issue. People were complaining about how long stuf had been waiting. I'm not imagining that, pretty sure. More generally, there are

[quagga-dev 16158] Re: Anyone at netdev conference in Tokyo?

2016-10-03 Thread Vincent Jardin
I am. See you on Wednesday. Le 3 octobre 2016 11:33:58 "Martin Winter" a écrit : Any Quagga folks at the netdevconf (http://www.netdevconf.org/1.2/) in Tokyo? If yes, ping me - would be good to get together for a beer. - Martin Winter mwin...@opensourcerouting.org