Re: [ovs-dev] [PATCH v2 0/2] OVN DB log fixes

2017-11-28 Thread Numan Siddique
Thank you Ben for backports and for taking the trouble to apply the dependent patches. Next time I make sure to apply locally before requesting for the branches. Thanks Numan On Tue, Nov 28, 2017 at 12:47 AM, Ben Pfaff wrote: > OK, I did these backports. > > For branch-2.7 I had to backport al

Re: [ovs-dev] [PATCH v2 0/2] OVN DB log fixes

2017-11-27 Thread Ben Pfaff
OK, I did these backports. For branch-2.7 I had to backport all of the following commits to make these patches apply cleanly: commit 2abbe32153b7e4719b39f477b35e7cc40231338a Author: Numan Siddique Date: Wed Nov 8 14:28:49 2017 +0530 ovn-ctl: Add -vfile:info option to OVN_NB/SB_LOG options

Re: [ovs-dev] [PATCH v2 0/2] OVN DB log fixes

2017-11-26 Thread Numan Siddique
Hi Ben, Thanks for the review and applying the patches. The issue is seen with 2.7 branch. So It would be great if it is backported to 2.8 and 2.7 branches. Thanks again Numan On Mon, Nov 27, 2017 at 6:02 AM, Ben Pfaff wrote: > I applied this series to master. Let me know if you want backp

Re: [ovs-dev] [PATCH v2 0/2] OVN DB log fixes

2017-11-26 Thread Ben Pfaff
I applied this series to master. Let me know if you want backports. On Wed, Nov 08, 2017 at 02:28:13PM +0530, nusid...@redhat.com wrote: > From: Numan Siddique > > v1 -> v2 > --- > In patch 2, changed the approach. Instead of fixing the issue in > ovs-appctl, corrected the ovs logrotate scr

[ovs-dev] [PATCH v2 0/2] OVN DB log fixes

2017-11-08 Thread nusiddiq
From: Numan Siddique v1 -> v2 --- In patch 2, changed the approach. Instead of fixing the issue in ovs-appctl, corrected the ovs logrotate script to use complete unix ctl file path as suggested by Ben. No changes to patch 1. v1 -- In Openstack deployment with OVN HA (with v2.7.2) in RHEL,