Thanks Lucas for the patch: It's good to update inactivity_probe via crm resource as we ideally use ovn-nbtl/ovn-sbctl commands to do that since the value one chooses depends on scale and can be different for everyone and we always tune it from active node. Also what do you mean by multiple connections? It's possible to run ovn nb db and ovn sb db in dual mode with pssl and ptcp which is where you can set multiple entries in the connection table afaik and sure instead of running nbctl/sbctl multiple times, this patch will help. Hence, brief explanation in doc would be helpful too. Since you are exposing this through crm_resource, update Documentation/topics/integration.rst in pacemaker section so that user can also read that option and leverage multiple probe update via this patch.
On Fri, Jun 28, 2019 at 10:08 PM Numan Siddique <[email protected]> wrote: > On Thu, Jun 27, 2019 at 12:43 AM Numan Siddique <[email protected]> > wrote: > > > > > > > On Thu, Jun 27, 2019 at 12:41 AM Ben Pfaff <[email protected]> wrote: > > > >> On Wed, Jun 26, 2019 at 11:59:30AM +0100, [email protected] wrote: > >> > From: Lucas Alvares Gomes <[email protected]> > >> > > >> > This patch is enhacing the ovndb-servers.ocf script to handle updates > to > >> > the inactive_probe_interval via pacemaker. For example, one could run: > >> > > >> > $ sudo crm_resource --resource ovndb_servers --set-parameter > >> > inactive_probe_interval --parameter-value <new probe value> > >> > > >> > To set a new inactive probe interval in OVSDB. The patch also handles > >> > the case were multiple connection exists. > >> > > >> > Signed-off-by: Lucas Alvares Gomes <[email protected]> > >> > > > Acked-by: Numan Siddique <[email protected]> > > > > > > >> Numan or Ali, would you mind reviewing this? It is outside my expertise > >> and you seem to be recent contributors. > >> > > > > I will take a look at it. > > > > Thanks > > Numan > > > > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
