From 581495082a5a0a9456065b3d3bb8b5f015747fd8 Mon Sep 17 00:00:00 2001
From: Eric MacDonald <eric.macdonald@windriver.com>
Date: Tue, 6 Apr 2021 09:02:39 -0400
Subject: [PATCH] Make small modification to fm's logrotation configuration
 file

This update makes the following changes to the fm logrotation config file

 - add 'create' with permissions to each tuple
 - add 'delaycompress' as a local setting to each log entry
 - remove 'nodateext' global and local setting

Test Plan:

PASS: Verify fm logs rotation behavior
PASS: Verify fm logs delaycompress setting behavior
PASS: Verify log permissions after rotate

Change-Id: Ibe8bd8107501df947b5091e928de202378ef4ea8
Partial-Bug: 1918979
Depends-On: https://review.opendev.org/c/starlingx/config-files/+/784943
Signed-off-by: Eric MacDonald <eric.macdonald@windriver.com>
---
 fm-mgr/sources/fm.logrotate | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/fm-mgr/sources/fm.logrotate b/fm-mgr/sources/fm.logrotate
index 486f3806..99d6714b 100644
--- a/fm-mgr/sources/fm.logrotate
+++ b/fm-mgr/sources/fm.logrotate
@@ -3,25 +3,25 @@
 #
 # SPDX-License-Identifier: Apache-2.0
 #
-nodateext
 
 /var/log/fm-event.log
 {
-    nodateext
+    create 0640 root root
     size 100M
     start 1
-    missingok
     rotate 20
+    missingok
     compress
     sharedscripts
     postrotate
         systemctl reload syslog-ng > /dev/null 2>&1 || true
-    endscript 
+    endscript
+    delaycompress
 }
 
 /var/log/fm-manager.log
 {
-    nodateext
+    create 0600 root root
     size 10M
     start 1
     rotate 20
@@ -32,4 +32,5 @@ nodateext
     postrotate
         systemctl reload syslog-ng > /dev/null 2>&1 || true
     endscript
+    delaycompress
 }