Merge pull request #64 from phaedriel/usergroup

manifests/config.pp

@@ -14,8 +14,8 @@
   file { '/etc/kibana/kibana.yml':
     ensure  => $_ensure,
     content => template("${module_name}/etc/kibana/kibana.yml.erb"),
-    owner   => 'kibana',
-    group   => 'kibana',
+    owner   => $kibana::kibana_user,
+    group   => $kibana::kibana_group,
     mode    => '0660',
   }
 }

manifests/init.pp

@@ -22,6 +22,8 @@
 # @param package_source Local path to package file for file (not repo) based installation
 # @param manage_repo Whether to manage the package manager repository
 # @param status Service status
+# @param kibana_user owner of kibana.yml
+# @param kibana_group group of kibana.yml
 #
 # @author Tyler Langlois <tyler.langlois@elastic.co>
 #
@@ -32,6 +34,8 @@
   Boolean $oss,
   Optional[String] $package_source,
   Kibana::Status $status,
+  String[1] $kibana_user = 'kibana',
+  String[1] $kibana_group = 'kibana',
 ) {
   contain kibana::install
   contain kibana::config

spec/classes/kibana_spec.rb

@@ -172,6 +172,18 @@
             end
           end
 
+          describe 'kibana_user' do
+            let(:params) { { kibana_user: 'testuser' } }
+
+            it { is_expected.to contain_file('/etc/kibana/kibana.yml').with(owner: 'testuser') }
+          end
+
+          describe 'kibana_group' do
+            let(:params) { { kibana_group: 'testgroup' } }
+
+            it { is_expected.to contain_file('/etc/kibana/kibana.yml').with(group: 'testgroup') }
+          end
+
           describe 'manage_repo' do
             let(:params) { { manage_repo: false } }