[ovs-git] [openvswitch/ovs] fefb75: debian, xenserver: Update logrotate config to matc...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: fefb757ce408a17f95d8a0c509d5796da45f75fe
      https://github.com/openvswitch/ovs/commit/fefb757ce408a17f95d8a0c509d5796da45f75fe
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2017-04-24 (Mon, 24 Apr 2017)

  Changed paths:
    M debian/openvswitch-switch.logrotate
    M xenserver/etc_logrotate.d_openvswitch

  Log Message:
  -----------
  debian, xenserver: Update logrotate config to match RHEL.

Commit 618a5b45ae8b ("rhel: Avoid logrotate error if /var/run/openvswitch
does not exist") updated the RHEL logrotate configuration.  This commit
makes similar changes for Debian, by synchronizing with the RHEL version.

In particular:

    - Indent to match logrotate.conf(5) examples.

    - Use "sharedscripts" flag, because the postrotate script only needs to
      run once regardless of the number of rotations.

    - Drop "delaycompress", because the postrotate script does make daemons
      reopen their log files.

    - Ignore errors calling vlog/reopen.

Also make similar changes to the xenserver logrotate script.  I confirmed
via Twitter that the xenserver packaging still has users.

CC: Timothy Redaelli <tredaelli at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Gurucharan Shetty <guru at ovn.org>