Re: [Bsdrp-users] BSDEP 1.992 issues with FRR

[Lyubomir Y. <l....@gm...>]

Hi Olivier,
Do you plan to upload a newer version with fixed FRR?
Regards,
Lyubo

On Wed, 7 Feb 2024 at 17:04, Lyubomir Yotov <l....@gm...> wrote:

> Hi Olivier,
> Thanks for the feedback. I fully agree with your point on the “reload”
> option.
> Looking forward to the next release.
>
> Regards,
> Lyubo
>
>
> On Wed, 7 Feb 2024 at 15:42, Olivier Cochard-Labbé <ol...@co...>
> wrote:
>
>>
>>
>> On Thu, Feb 1, 2024 at 8:15 AM Lyubomir Yotov <l....@gm...> wrote:
>>
>>> Hello Olivier,
>>> I tried version 1.992 and besides the startup script for staticd issue
>>> (FreeBSD bugzilla bug 276333) I had two other issues:
>>>
>>> 1. When restarting the service with the patched startup script (as per
>>> the patch from bug 276333) the mgmtd dawmon doesn’t start (below is the
>>> output). Since FRR 9.0 a centralized management daemon (mgmtd) is
>>> introduced and staticd configuration is migrated to mgmtd. Is there a way
>>> fir workaround?
>>>
>>>
>>>
>> Yes, FreeBSD's port tree was updated to fix that.
>>
>>
>>> 2. In the configuration of route maps it is not possible to add prefix
>>> lists because of
>>> https://github.com/FRRouting/frr/issues/14375
>>> I saw that the libyang version in 1.992 is 2.1.111
>>> Have you tested with libyand 2.1.80?
>>> Can you please provide an updated version with correctly patched frr
>>> service script and with libyang 2.1.80 if you find appropriate?
>>>
>>
>> I’ve read the exchange between FRR and libyang teams, they are
>> interesting regarding the definition of a stable API :-)
>>
>> libyang 2.1.128, that patch this regression, was committed in FreeBSD's
>> port tree.
>> https://github.com/CESNET/libyang/releases/tag/v2.1.128
>>
>>
>> So I just need to release a new BSDRP release with this updated port tree
>> to fix those 2 problems.
>>
>>
>>
>>> One more question, is it possible to have the option “service frr
>>> reload”?
>>>
>>
>> There is a dedicated python tool (in net/frr9-pythontools
>> as sbin/frr-reload.py) for this feature.
>> I didn’t look at it, but my thought was «If they need a python script
>> with more than 2000 lines for this reload feature, it will be a nightmare
>> to do it inside the RC shell script».
>> This is why there is no such "reload" feature currently: I hope once all
>> daemons being converted to use mgmtd, this will be a feature it will take
>> care of.
>>
>> Regards,
>> Olivier
>>
>>