Merge pull request #615 from Kristijan/logfile_user_group_perms

Allow setting user and group permissions of the NTP logfile

Author: carabasdaniel

data/common.yaml

@@ -26,7 +26,9 @@ ntp::keys_trusted: []
 ntp::keys: []
 ntp::leapfile: ~
 ntp::logfile: ~
+ntp::logfile_user: 'ntp'
 ntp::logfile_mode: '0664'
+ntp::logfile_group: 'ntp'
 ntp::logconfig: ~
 ntp::ntpsigndsocket: ~
 ntp::maxpoll: ~

manifests/config.pp

@@ -139,8 +139,8 @@
   if $ntp::logfile {
     file { $ntp::logfile:
       ensure => file,
-      owner  => 'ntp',
-      group  => 'ntp',
+      owner  => $ntp::logfile_user,
+      group  => $ntp::logfile_group,
       mode   => $ntp::logfile_mode,
     }
   }

manifests/init.pp

@@ -87,10 +87,16 @@
 #
 # @param logfile
 #   Specifies a log file for NTP to use instead of syslog. Default value: ' '.
-
+#
+# @param logfile_group
+#   Specifies the group for the NTP log file. Default is 'ntp'.
+#
 # @param logfile_mode
 #   Specifies the permission for the NTP log file. Default is 0664.
 #
+# @param logfile_user
+#   Specifies the user for the NTP log file. Default is 'ntp'.
+#
 # @param logconfig
 #   Specifies the logconfig for NTP to use. Default value: ' '.
 #
@@ -248,7 +254,9 @@
   Stdlib::Absolutepath $driftfile,
   Optional[Stdlib::Absolutepath] $leapfile,
   Optional[Stdlib::Absolutepath] $logfile,
+  Optional[Variant[String, Integer]] $logfile_group,
   String $logfile_mode,
+  Optional[Variant[String, Integer]] $logfile_user,
   Optional[String] $logconfig,
   Boolean $iburst_enable,
   Array[String] $keys,