Merge "Remove deprecated aodh::evaluator::coordination_url"

manifests/coordination.pp

@@ -32,14 +32,8 @@ class aodh::coordination (
 
   include aodh::deps
 
-  if defined('$::aodh::evaluator::coordination_url') {
-    $backend_url_real = pick($::aodh::evaluator::coordination_url, $backend_url)
-  } else {
-    $backend_url_real = $backend_url
-  }
-
   oslo::coordination{ 'aodh_config':
-    backend_url => $backend_url_real
+    backend_url => $backend_url
   }
 
   aodh_config {

manifests/evaluator.pp

@@ -30,12 +30,6 @@
 #    compensate the reporting/ingestion lag.
 #    Defaults to $::os_service_default.
 #
-# DEPRECATED PARAMETERS
-#
-#  [*coordination_url*]
-#    (optional) The url to use for distributed group membership coordination.
-#    Defaults to undef.
-#
 class aodh::evaluator (
   $manage_service           = true,
   $enabled                  = true,
@@ -44,18 +38,11 @@ class aodh::evaluator (
   $evaluation_interval      = $::os_service_default,
   $event_alarm_cache_ttl    = $::os_service_default,
   $additional_ingestion_lag = $::os_service_default,
-  # DEPRECATED PARAMETERS
-  $coordination_url         = undef,
 ) {
 
   include aodh::deps
   include aodh::params
 
-  if $coordination_url != undef {
-    warning('The coordination_url parameter is deprecated. Use the aodh::coordination class instead')
-    include aodh::coordination
-  }
-
   aodh_config {
     'evaluator/evaluation_interval':    value => $evaluation_interval;
     'DEFAULT/event_alarm_cache_ttl':    value => $event_alarm_cache_ttl;

releasenotes/notes/cleanup-coordination-ca6319f55bf4fdb2.yaml

@@ -0,0 +1,4 @@
+---
+upgrade:
+  - |
+    The ``aodh::evaluator::coordination_url`` parameter has been removed.

spec/classes/aodh_evaluator_spec.rb

@@ -42,16 +42,6 @@ describe 'aodh::evaluator' do
       end
     end
 
-    context 'with deprecated coordination_url' do
-      before do
-        params.merge!({ :coordination_url => 'redis://localhost:6379' })
-      end
-      it 'configures coordination and workers' do
-        is_expected.to contain_aodh_config('coordination/backend_url').with_value('redis://localhost:6379')
-        is_expected.to contain_aodh_config('evaluator/workers').with_value(4)
-      end
-    end
-
     context 'when enabled' do
       it { is_expected.to contain_class('aodh::params') }