Re: [ovs-dev] [PATCH v2] Make pid_exists() more robust against empty pid argument

2019-08-30 Thread Ilya Maximets
On 29.08.2019 17:46, Ben Pfaff wrote: > On Tue, Aug 27, 2019 at 02:43:21PM +0300, Ilya Maximets wrote: >> On 14.08.2019 18:47, Michele Baldessari wrote: >>> In some of our destructive testing of ovn-dbs inside containers managed >>> by pacemaker we reached a situation where /var/run/openvswitch

Re: [ovs-dev] [PATCH v2] Make pid_exists() more robust against empty pid argument

2019-08-29 Thread Ben Pfaff
On Tue, Aug 27, 2019 at 02:43:21PM +0300, Ilya Maximets wrote: > On 14.08.2019 18:47, Michele Baldessari wrote: > > In some of our destructive testing of ovn-dbs inside containers managed > > by pacemaker we reached a situation where /var/run/openvswitch had > > empty .pid files. The current code

Re: [ovs-dev] [PATCH v2] Make pid_exists() more robust against empty pid argument

2019-08-27 Thread Ilya Maximets
On 14.08.2019 18:47, Michele Baldessari wrote: > In some of our destructive testing of ovn-dbs inside containers managed > by pacemaker we reached a situation where /var/run/openvswitch had > empty .pid files. The current code does not deal well with them > and pidfile_is_running() returns true in

[ovs-dev] [PATCH v2] Make pid_exists() more robust against empty pid argument

2019-08-14 Thread Michele Baldessari
In some of our destructive testing of ovn-dbs inside containers managed by pacemaker we reached a situation where /var/run/openvswitch had empty .pid files. The current code does not deal well with them and pidfile_is_running() returns true in such a case and this confuses the OCF resource agent.