[quagga-dev 12735] Re: [PATCH] realms support and small fix
sharpd at cumulusnetworks.com
Mon Jun 15 17:58:01 BST 2015
Possibly need to break up the patch into smaller patches, starting with
changes to lib, then zebra, then bgp.
Why does bgp display the realm id, not the the realm name?
Why is support for realms bgp only? Shouldn't it just be part of the
normal route-map code in lib/routemap.c?( I may be wrong here, I can
safely say I don't fully understand the routemap code yet )
Is there any particular reason we reread /etc/iproute2/rt_realms file
for every call into rtnl_rtrealm_a2n()?
The expected file format for /etc/iproute2/rt_realms is pretty limiting,
is this intentional?( Who maintains this file? By hand? )
I think I would prefer to just add the u_int16_t realmid to rib_add_ipv4
function and ignore the realmid passed in if it's not used. I'm not a
big fan of functions with different #'s of parameters based upon
configure options. I would personally prefer the lower level function
just do the 'right' thing if it's configured and do the right thing if
On 6/15/15 10:24 AM, Kaloyan Kovachev wrote:
> Hello devs,
> I am using the realms patch from Adrian Ban  for several years, but
> it is still missing from Quagga, while it is very useful (not only for
> me I hope).
> I have ported the patch for the latest version on GitHup and while
> working on it found another small bug, which I have fixed 
> Is there something else that needs to be done so they can be included
> in the official source?
>  http://linux.mantech.ro/quagga+realm_en.php
> Quagga-dev mailing list
> Quagga-dev at lists.quagga.net
More information about the Quagga-dev