diff --git a/manifests/agent/polling.pp b/manifests/agent/polling.pp index e7b70102..608781d6 100644 --- a/manifests/agent/polling.pp +++ b/manifests/agent/polling.pp @@ -115,16 +115,8 @@ class ceilometer::agent::polling ( if $central_namespace { $central_namespace_name = 'central' - ceilometer_config { - # set `tenant_name_discovery` parameter only on the nodes - # where central namespace is enabled - 'DEFAULT/tenant_name_discovery': value => $tenant_name_discovery; - } } else { $central_namespace_name = undef - ceilometer_config { - 'DEFAULT/tenant_name_discovery': ensure => absent; - } } if $compute_namespace { @@ -225,7 +217,8 @@ class ceilometer::agent::polling ( } ceilometer_config { - 'polling/batch_size': value => $batch_size + 'polling/batch_size': value => $batch_size; + 'DEFAULT/tenant_name_discovery': value => $tenant_name_discovery; } if $manage_service { diff --git a/releasenotes/notes/fix_tenant_name_discovery-838f5d64bc405eca.yaml b/releasenotes/notes/fix_tenant_name_discovery-838f5d64bc405eca.yaml new file mode 100644 index 00000000..8c407505 --- /dev/null +++ b/releasenotes/notes/fix_tenant_name_discovery-838f5d64bc405eca.yaml @@ -0,0 +1,6 @@ +--- +fixes: + - | + Configure ``tenant_name_discovery`` parameter on + all nodes where ceilometer is expected to run. This inculdes + central, compute and ipmi namespaces. diff --git a/spec/classes/ceilometer_agent_polling_spec.rb b/spec/classes/ceilometer_agent_polling_spec.rb index 21050518..5543fb63 100644 --- a/spec/classes/ceilometer_agent_polling_spec.rb +++ b/spec/classes/ceilometer_agent_polling_spec.rb @@ -38,7 +38,6 @@ describe 'ceilometer::agent::polling' do )} it { - should contain_ceilometer_config('DEFAULT/tenant_name_discovery').with_value('') should contain_ceilometer_config('compute/instance_discovery_method').with_value('') should contain_ceilometer_config('compute/resource_update_interval').with_value('') should contain_ceilometer_config('compute/resource_cache_expiry').with_value('') @@ -73,6 +72,7 @@ describe 'ceilometer::agent::polling' do it { should contain_ceilometer_config('polling/batch_size').with_value('') } it { should_not contain_file('polling') } + it { should contain_ceilometer_config('DEFAULT/tenant_name_discovery').with_value('') } end context 'when setting package_ensure' do @@ -101,15 +101,15 @@ describe 'ceilometer::agent::polling' do } end - context 'with central parameters set' do + context 'when tenant_name_discovery is set' do before do params.merge!( - :tenant_name_discovery => false + :tenant_name_discovery => true ) end it { - should contain_ceilometer_config('DEFAULT/tenant_name_discovery').with_value(false) + should contain_ceilometer_config('DEFAULT/tenant_name_discovery').with_value(true) } end @@ -137,7 +137,6 @@ describe 'ceilometer::agent::polling' do it { should contain_ceilometer_config('DEFAULT/polling_namespaces').with_value('compute,ipmi') - should contain_ceilometer_config('DEFAULT/tenant_name_discovery').with_ensure('absent') } end