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 <[email protected]> wrote: > I applied this series to master. Let me know if you want backports. > > On Wed, Nov 08, 2017 at 02:28:13PM +0530, [email protected] wrote: > > From: Numan Siddique <[email protected]> > > > > 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, we see that > > OVN DB log files are empty after deployment. Adding "-vfile:info" option > > when starting ovsdb-servers fixes this issue. Another issue seen is when > > openvswitch logrotate script [1] is called, it doesn't initialize the > log files > > for the OVN DB servers because of which the log file is empty. > > > > This patch series fixes these issue. > > > > It would be good if these fixes are applied to branches 2.8 and 2.7. > > > > [1] - https://github.com/openvswitch/ovs/blob/master/ > rhel/etc_logrotate.d_openvswitch > > > > > > Numan Siddique (2): > > ovn-ctl: Add -vfile:info option to OVN_NB/SB_LOG options > > OpenvSwitch logrotate: Use ctl file path as target in ovs-appctl to > > reset logs > > > > debian/openvswitch-switch.logrotate | 6 +++--- > > ovn/utilities/ovn-ctl | 4 ++-- > > rhel/etc_logrotate.d_openvswitch | 4 ++-- > > 3 files changed, 7 insertions(+), 7 deletions(-) > > > > -- > > 2.13.5 > > > > _______________________________________________ > > 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
