[ovs-dev] [PATCH ovn] ovn-northd: Add the option to configure probe interval

Numan Siddique numans at ovn.org
Thu Mar 5 06:54:27 UTC 2020


On Thu, Mar 5, 2020 at 11:40 AM Girish Moodalbail
<gmoodalbail at nvidia.com> wrote:
>
> Hello Numan,
>
> On 3/4/20, 9:58 PM, "Numan Siddique" <numans at ovn.org> wrote:
>
>     External email: Use caution opening links or attachments
>
>
>     On Thu, Mar 5, 2020 at 10:22 AM Girish Moodalbail
>     <gmoodalbail at nvidia.com> wrote:
>     >
>     > Hello Numan,
>     >
>     > Thank you for getting this done __.
>     >
>     > Quick comment: The option set on NB_Global also affects ovn-northd's connection to OVN SB as well. Typically, one would expect any option on NB_Global should affect ovsdb-server managing OVN NB DB alone, right? Should we add similar option to SB_Global as well?
>
>     Hi Girish,
>
>     This newly added option probe_interval (which will be renamed to
>     northd_probe_interval) applies to both Northbound and Southbound
>     connections from ovn-northd.
>     So there is no need to add in SB_Global. And CMS is not supposed to do
>     any write txns to Southbound db.
>
> I see. If that is the case, then one option should be fine then.

Thanks  Han and Girish for the review and comments.

I applied this patch to master with the changes suggested by Han.

Thanks
Numan

>
> Regards,
> ~Girish
>
>
>
> -----------------------------------------------------------------------------------
> This email message is for the sole use of the intended recipient(s) and may contain
> confidential information.  Any unauthorized review, use, disclosure or distribution
> is prohibited.  If you are not the intended recipient, please contact the sender by
> reply email and destroy all copies of the original message.
> -----------------------------------------------------------------------------------
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev


More information about the dev mailing list