On Fri, May 15, 2015 at 12:10:06PM -0700, Vipin Kumar wrote: > However, I would like to point out (a late finding) that the command mode > for this feature isn't quite right for the current BGP code/feature set. [...] > > (A) > Make this feature truly global to BGP deamon by moving the update-delay > config at the level where 'bgp multi-instance' is configured. > [...] > OR > > (B) > Keep the config where it is, but fix the implementation to make it per view. [...]
I'd argue we should start with A, i.e. fix it up into a consistent state, and then work on "more than B": make it per-view *and* per-AF. Making it per-AF means we can exit update-delay earlier for AFs that process quicker - like IPv6. That'd also mean we run the IPv6 batch earlier, and have less of a big pile to work through later when IPv4/everything is ready. (It also lines up with Graceful Restart behaviour) -David _______________________________________________ Quagga-dev mailing list [email protected] https://lists.quagga.net/mailman/listinfo/quagga-dev
