On Tue, May 23, 2017 at 5:21 AM, Andy Zhou <[email protected]> wrote: > On Sun, May 21, 2017 at 6:35 PM, <[email protected]> wrote: > > From: Numan Siddique <[email protected]> > > > > start action returns OCF_RUNNING_MASTER in certain scenarios. > > But as per the OCF guidelines, status code OCF_RUNNING_MASTER shoud > > be returned only in monitor action [1]. > > > > Whenever the start action returns OCF_RUNNING_MASTER, it is observed > > in the testing that, pacemaker stops the ovsdb-server ocf resource > > in that node. This patch fixes this issue by returning OCF_SUCESS in > > such cases. > > > > stop action returns OCF_RUNNING_MASTER if the ovsdb-servers are > > running as master. But as per the OCF guidelines [2], stop action > > should only return OCF_SUCCESS. If any other code is returned, > > pacemaker cluster would block that resource in that node. > > > > This patch fixes this issue by stopping the ovsdb-servers when they > > are running as masters (which is the expected case) and returns > > OCF_SUCCESS. > > > > [1] - http://www.linux-ha.org/doc/dev-guides/_literal_ocf_ > running_master_literal_8.html > > [2] - http://www.linux-ha.org/doc/dev-guides/_literal_stop_ > literal_action.html > > > > CC: Andy Zhou <[email protected]> > > Signed-off-by: Numan Siddique <[email protected]> > > Thanks for the fixes! Both patches look reasonable to me. I pushed > them to master. >
Thanks Andy. Can these patches be back ported to branch 2.7 ? It would be great since the tripleo patches for OVN needs these fixes Numan _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
