[ovs-dev] [PATCH ovn] Fix connection string in case of changes in the ovndb-servers.ocf RA

2021-03-25 Thread Michele Baldessari
vsdb-server",pid=516770,fd=15)) ino:11641860 sk:4d <-> tcp LISTEN 0 104.5.6.7:6642 0.0.0.0:* users:(("ovsdb-server",pid=516789,fd=15)) ino:11642885 sk:4e <-> Previously we'd observe ovsdb-server listening to the old IP even after the resource ch

[ovs-dev] [PATCH ovn v2] Make the notify() calls work with IPv6 in the OCF resource-agent

2020-06-03 Thread Michele Baldessari
the create Connection command was already escaping the columns, we are just missing the ip address bits in case of IPv6. Let's make sure we escape the '[]:' characters and avoid this problem. Tested this on an OpenStack environment on both IPv6 and IPv4. Signed-off-by: Michele Baldessari --- utili

Re: [ovs-dev] [PATCH] Make the notify() calls work with IPv6

2020-06-03 Thread Michele Baldessari
On Sun, May 31, 2020 at 12:19:07PM +0100, Gabriele Cerami wrote: > AFAIK OVN patches should have PATCH ovn prefix :) Ah ok, I guess last time I tweaked the OCF resource agent it was still in the ovs repo. > On 31 May, Michele Baldessari wrote: > > --- > > utilities/ovn

[ovs-dev] [PATCH] Make the notify() calls work with IPv6

2020-05-31 Thread Michele Baldessari
the create Connection command was already escaping the columns, we are just missing the ip address bits in case of IPv6. Let's make sure we escape the '[]:' characters and avoid this problem. Tested this on an OpenStack environment on both IPv6 and IPv4. Signed-off-by: Michele Baldessari --- utili

Re: [ovs-dev] [PATCH] Make pidfile_is_running more robust against empty pidfiles

2019-08-14 Thread Michele Baldessari
On Wed, Aug 14, 2019 at 02:28:13PM +0300, Ilya Maximets wrote: > On 14.08.2019 11:39, 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 fi

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

2019-08-14 Thread Michele Baldessari
-lib: Allow "status" command to work as non-root.") Signed-off-by: Michele Baldessari --- v1 -> v2 - Implemented Ilya's suggestion and moved the check from pidfile_is_running() to pid_exists() and re-run my tests --- utilities/ovs-lib.in | 2 +- 1 file changed, 1 inse

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

2019-08-14 Thread Michele Baldessari
-lib: Allow "status" command to work as non-root.") Signed-off-by: Michele Baldessari --- v1 -> v2 - Implemented Ilya's suggestion and moved the check from pidfile_is_running() to pid_exists() and re-run my tests --- utilities/ovs-lib.in | 2 +- 1 file changed, 1 inse

[ovs-dev] [PATCH] Make pidfile_is_running more robust against empty pidfiles

2019-08-14 Thread Michele Baldessari
:ovndb-servers): Master controller-0 ovn-dbs-bundle-1 (ocf::ovn:ovndb-servers): Slave controller-1 ovn-dbs-bundle-2 (ocf::ovn:ovndb-servers): Slave controller-2 Signed-off-by: Michele Baldessari --- ovn/utilities/ovn-ctl | 2 +- 1 file changed, 1 insertion(+), 1 de

Re: [ovs-dev] [PATCH] OVN resource agent - make promotion synchronous

2019-07-17 Thread Michele Baldessari
can still fix it up if that is preferred? cheers, Michele > > On Tue, Jul 9, 2019 at 3:10 AM 0-day Robot wrote: > > > Bleep bloop. Greetings Michele Baldessari, I am a robot and I have tried > > out your patch. > > Thanks for your contribution. > > > >

[ovs-dev] [PATCH] OVN resource agent - make promotion synchronous

2019-07-09 Thread Michele Baldessari
-off-by: Michele Baldessari --- ovn/utilities/ovndb-servers.ocf | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/ovn/utilities/ovndb-servers.ocf b/ovn/utilities/ovndb-servers.ocf index 10313304cb7c..cd47426689ef 100755 --- a/ovn/utilities/ovndb-servers.ocf +++ b/ovn