On Sat, Oct 06, 2018 at 08:04:09PM +0530, nusid...@redhat.com wrote:
> From: Numan Siddique <nusid...@redhat.com>
>
> With this commit the users can override the default values of
> the BFD params - min_rx, min_tx, decay_min_rx and mult if desired.
> This can be useful to debug any issues related to BFD (like
> frequent BFD state changes).
>
> A new column 'options' is added in NB_Global and SB_Global tables
> of OVN_Northbound and OVN_Southbound schemas respectively. CMS
> can define the options 'bfd:min-rx', 'bfd:min-tx',
> 'bfd:decay-min-rx' and 'bfd:mult' in the options column of
> NB_Global table row. ovn-northd copies these options from
> NB_Global to SB_Global. ovn-controller configures these
> options to the tunnel interfaces when enabling BFD.
>
> Signed-off-by: Numan Siddique <nusid...@redhat.com>
Thank you for working to make OVN easier to debug and troubleshoot.
Using : in the bfd option names makes it difficult to set them using
ovn-nbctl. It is necessary to understand how to escape the :, as shown
in the test, e.g.:
ovn-nbctl --wait=hv set NB_Global . options:"bfd\:min-rx"=2000
Do you think it would be a good idea to use - instead of : for this
reason?
When bfd_run() decides to disable bfd on an interface, it looks like it
will still add any user-specified BFD settings. Is that intentional?
The function interface_set_bfd() is now trivial, so it might make sense
to inline it into the caller.
Thanks,
Ben.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
