new parameters logging support

Author: stjmt

REFERENCE.md

@@ -131,6 +131,7 @@ The following parameters are available in the `elasticsearch` class:
 * [`logdir_mode`](#-elasticsearch--logdir_mode)
 * [`manage_datadir`](#-elasticsearch--manage_datadir)
 * [`manage_logdir`](#-elasticsearch--manage_logdir)
+* [`manage_logging`](#-elasticsearch--manage_logging)
 * [`manage_repo`](#-elasticsearch--manage_repo)
 * [`oss`](#-elasticsearch--oss)
 * [`package_dir`](#-elasticsearch--package_dir)
@@ -410,6 +411,14 @@ Data type: `Boolean`
 
 Enable logdir management (default true).
 
+##### <a name="-elasticsearch--manage_logging"></a>`manage_logging`
+
+Data type: `Boolean`
+
+Enable logging (log4j) management (default false).
+
+Default value: `false`
+
 ##### <a name="-elasticsearch--manage_repo"></a>`manage_repo`
 
 Data type: `Boolean`

manifests/config.pp

@@ -141,6 +141,20 @@
       $_tls_config = {}
     }
 
+    # Logging file
+    if $elasticsearch::manage_logging and !empty($elasticsearch::logging_content) {
+      file { $elasticsearch::logging_path:
+        ensure  => file,
+        content => $elasticsearch::logging_content,
+        group   => $elasticsearch::elasticsearch_group,
+        owner   => $elasticsearch::elasticsearch_user,
+        mode    => '0644',
+        notify  => $elasticsearch::_notify_service,
+        require => File[$elasticsearch::configdir],
+        before  => Class['elasticsearch::service'],
+      }
+    }
+
     # Generate Elasticsearch config
     $data =
       $elasticsearch::config + { 'path.data' => $elasticsearch::datadir } + { 'path.logs' => $elasticsearch::logdir } + $_tls_config

manifests/init.pp

@@ -139,6 +139,9 @@
 # @param manage_logdir
 #   Enable logdir management (default true).
 #
+# @param manage_logging
+#   Enable logging (log4j) management (default false).
+#
 # @param manage_repo
 #   Enable repo management by enabling official Elastic repositories.
 #
@@ -375,6 +378,9 @@
   Optional[String]                                $keystore_password         = undef,
   Optional[Stdlib::Absolutepath]                  $keystore_path             = undef,
   Stdlib::Filemode                                $logdir_mode               = '2750',
+  Optional[String]                                $logging_content           = undef,
+  Stdlib::Absolutepath                            $logging_path              = "${configdir}/log4j2.properties",
+  Boolean                                         $manage_logging            = false,
   Boolean                                         $package_hold              = false,
   Optional[Stdlib::Absolutepath]                  $private_key               = undef,
   Enum['rsa','dsa','ec']                          $private_key_type          = 'rsa',

spec/classes/000_elasticsearch_init_spec.rb

@@ -528,6 +528,59 @@
         }
       end
 
+      context 'When managing the logging file (with content)' do
+        let(:params) do
+          default_params.merge(
+            logging_content: '# Content',
+            manage_logging: true
+          )
+        end
+
+        it {
+          expect(subject).to contain_file('/etc/elasticsearch/log4j2.properties').
+            with(ensure: 'file', content: '# Content')
+        }
+      end
+
+      context 'When managing the logging file (with content and specific path)' do
+        let(:params) do
+          default_params.merge(
+            logging_content: '# Content',
+            logging_path: '/etc/elasticsearch/log4j.properties',
+            manage_logging: true
+          )
+        end
+
+        it {
+          expect(subject).to contain_file('/etc/elasticsearch/log4j.properties').
+            with(ensure: 'file', content: '# Content')
+        }
+      end
+
+      context 'When managing the logging file (with no content)' do
+        let(:params) do
+          default_params.merge(
+            manage_logging: true
+          )
+        end
+
+        it {
+          expect(subject).not_to contain_file('/etc/elasticsearch/log4j2.properties')
+        }
+      end
+
+      context 'When not managing the logging file' do
+        let(:params) do
+          default_params.merge(
+            manage_logging: false
+          )
+        end
+
+        it {
+          expect(subject).not_to contain_file('/etc/elasticsearch/log4j2.properties')
+        }
+      end
+
       context 'with restart_on_change => true' do
         let(:params) do
           default_params.merge(