Use logstash_user and logstash_group instead of root

Phil Friderici · Phil Friderici · commit 512843b8625c · 2023-11-24T15:36:57.000Z
diff --git a/manifests/config.pp b/manifests/config.pp
@@ -9,8 +9,8 @@
   require logstash::package
 
   File {
-    owner => 'root',
-    group => 'root',
+    owner => $logstash::logstash_user,
+    group => $logstash::logstash_group,
   }
 
   # Configuration "fragment" directories for pipeline config and pattern files.
diff --git a/manifests/configfile.pp b/manifests/configfile.pp
@@ -49,7 +49,7 @@
 ) {
   include logstash
 
-  $owner = 'root'
+  $owner = $logstash::logstash_user
   $group = $logstash::logstash_group
   $mode  = '0640'
   $require = Package['logstash'] # So that we have '/etc/logstash/conf.d'.
diff --git a/manifests/patternfile.pp b/manifests/patternfile.pp
@@ -30,7 +30,7 @@
   file { "${logstash::config_dir}/patterns/${destination}":
     ensure => file,
     source => $source,
-    owner  => 'root',
+    owner  => $logstash::logstash_user,
     group  => $logstash::logstash_group,
     mode   => '0640',
     tag    => ['logstash_config'],
diff --git a/manifests/service.pp b/manifests/service.pp
@@ -37,8 +37,8 @@
   $pipelines = $logstash::pipelines
 
   File {
-    owner  => 'root',
-    group  => 'root',
+    owner  => $logstash::logstash_user,
+    group  => $logstash::logstash_group,
     mode   => '0644',
     notify => Exec['logstash-system-install'],
   }

Original file line number	Diff line number	Diff line change
`@@ -9,8 +9,8 @@`
`9`	`9`	`require logstash::package`
`10`	`10`
`11`	`11`	`File {`
`12`		`- owner => 'root',`
`13`		`- group => 'root',`
	`12`	`+ owner => $logstash::logstash_user,`
	`13`	`+ group => $logstash::logstash_group,`
`14`	`14`	`}`
`15`	`15`
`16`	`16`	`# Configuration "fragment" directories for pipeline config and pattern files.`