Drop time_to_live parameter

time_to_live was deprecated in liberty, let's drop it so we don't have
the warning anymore.

Change-Id: If36dc4ec2435aa8229b4efa6123fff9bdf668b37

diff --git a/examples/site.pp b/examples/site.pp
index e95812e2f53aed9f3f5bf8564b64d87d154f0ad0..d9b515867d032720db1520dd1bfe8816f925511f 100644 (file)
--- a/examples/site.pp
+++ b/examples/site.pp
@@ -82,10 +82,8 @@ node default {
   class { '::ceilometer::alarm::evaluator':
   }
 
-  # Purge 1 month old meters
-  class { '::ceilometer::expirer':
-    time_to_live => '2592000'
-  }
+  # Purge old meters
+  class { '::ceilometer::expirer': }
 
   # Install notification agent
   class { '::ceilometer::agent::notification':

diff --git a/manifests/expirer.pp b/manifests/expirer.pp
index edd518ecd0a8153ca76a6c0e0a1152d6ef9f730d..5168077468ef769484a10e343c2b9cdb01eac6c6 100644 (file)
--- a/manifests/expirer.pp
+++ b/manifests/expirer.pp
@@ -40,32 +40,20 @@
 #  [*weekday*]
 #    (optional) Defaults to '*'.
 #
-# [*time_to_live*]
-#   (optional) DEPRECATED. Number of seconds that samples are kept in the database.
-#   Should be a valid integer
-#   Defaults to '-1' to disable TTL and keep forever the datas.
 
 class ceilometer::expirer (
-  $enable_cron    = True,
-  $minute         = 1,
-  $hour           = 0,
-  $monthday       = '*',
-  $month          = '*',
-  $weekday        = '*',
-  # Deprecated parameters
-  $time_to_live   = '-1',
+  $enable_cron = True,
+  $minute      = 1,
+  $hour        = 0,
+  $monthday    = '*',
+  $month       = '*',
+  $weekday     = '*',
 ) {
 
   include ::ceilometer::params
 
   Package<| title == 'ceilometer-common' |> -> Class['ceilometer::expirer']
 
-  warning('Parameter "time_to_live" is deprecated and will be removed in next release. Use metering_time_to_live in "ceilometer" class instead.')
-
-  ceilometer_config {
-    'database/time_to_live': value => $time_to_live;
-  }
-
   if $enable_cron {
     cron { 'ceilometer-expirer':
       command     => $ceilometer::params::expirer_command,

diff --git a/spec/classes/ceilometer_expirer_spec.rb b/spec/classes/ceilometer_expirer_spec.rb
index 28f5525d29e2a7df81549d6d28776c60cff2affc..765e613f1ba2f52e7353fbb4bb7aa48be8dcad27 100644 (file)
--- a/spec/classes/ceilometer_expirer_spec.rb
+++ b/spec/classes/ceilometer_expirer_spec.rb
@@ -27,7 +27,7 @@ describe 'ceilometer::expirer' do
   end
 
   let :params do
-    { :time_to_live => '-1' }
+    {}
   end
 
   shared_examples_for 'ceilometer-expirer' do
@@ -62,10 +62,6 @@ describe 'ceilometer::expirer' do
       it { is_expected.to_not contain_cron('ceilometer-expirer') }
     end
 
-    it 'configures database section in ceilometer.conf' do
-      is_expected.to contain_ceilometer_config('database/time_to_live').with_value( params[:time_to_live] )
-    end
-
   end
 
   context 'on Debian platforms' do