]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Add purge_config option
authorZhongShengping <chdzsp@163.com>
Tue, 7 Jun 2016 02:10:06 +0000 (10:10 +0800)
committerZhongShengping <chdzsp@163.com>
Tue, 7 Jun 2016 03:20:37 +0000 (11:20 +0800)
Add the option in order to facilitate management.
User can ensure that only the options they hoped
are configured.

Change-Id: I149224f4035888e6bd2aca16c36af7ab9055a3a1

manifests/init.pp
spec/classes/ceilometer_init_spec.rb

index d81fc2870112d34ee90d9c97fb1c77072e26641d..e0a8ff6fe7f17b51b7835acfa87f6ead0d762cbe 100644 (file)
 #   (Optional) Password for message broker authentication
 #   Defaults to $::os_service_default.
 #
+# [*purge_config*]
+#   (optional) Whether to set only the specified config options
+#   in the ceilometer config.
+#   Defaults to false.
+#
 # === DEPRECATED PARAMETERS:
 #  [*metering_secret*]
 #   (optional)  Secret key for signing messages.
@@ -276,6 +281,7 @@ class ceilometer(
   $amqp_sasl_config_name              = $::os_service_default,
   $amqp_username                      = $::os_service_default,
   $amqp_password                      = $::os_service_default,
+  $purge_config                       = false,
   # DEPRECATED PARAMETERS
   $alarm_history_time_to_live         = undef,
   $metering_secret                    = undef,
@@ -324,6 +330,10 @@ class ceilometer(
     tag    => ['openstack', 'ceilometer-package'],
   }
 
+  resources { 'ceilometer_config':
+    purge => $purge_config,
+  }
+
   # we keep "ceilometer.openstack.common.rpc.impl_kombu" for backward compatibility
   if $rpc_backend in [$::os_service_default, 'ceilometer.openstack.common.rpc.impl_kombu', 'rabbit'] {
     oslo::messaging::rabbit {'ceilometer_config':
index 18b61dabf31d3456fa49a2e2f62489167b0a68e8..36af30d1f2a152cdc7e9557335029780f6d28e5d 100644 (file)
@@ -12,6 +12,7 @@ describe 'ceilometer' do
       :debug                      => 'False',
       :log_dir                    => '/var/log/ceilometer',
       :use_stderr                 => 'True',
+      :purge_config               => false,
     }
   end
 
@@ -105,6 +106,12 @@ describe 'ceilometer' do
       )
     end
 
+    it 'passes purge to resource' do
+      is_expected.to contain_resources('ceilometer_config').with({
+        :purge => false
+      })
+    end
+
     it 'configures required telemetry_secret' do
       is_expected.to contain_ceilometer_config('publisher/telemetry_secret').with_value('metering-s3cr3t')
       is_expected.to contain_ceilometer_config('publisher/telemetry_secret').with_value( params[:telemetry_secret] ).with_secret(true)