openstack/puppet-ceilometer
Code Issues Proposed changes

Remove store_events option

Browse Source 
Change-Id: I31e8c17a011320cacd3a6f9480452886f8df714b
This commit is contained in:
ZhongShengping 2018-02-07 14:15:37 +08:00 committed by zhongshengping
parent 41825ec5d4
commit c53c5424e2
2 changed files with 3 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
manifests/agent/notification.pp
View File

@ -73,12 +73,6 @@
# Defaults to ['gnocchi://'], If you are using collector
# override this to notifier:// instead.
#
# === DEPRECATED PARAMETERS:
# [*store_events*]
# (Optional) Save event details.
# This option has been removed since Newton.
#
class ceilometer::agent::notification (
$manage_service = true,
$enabled = true,
@ -91,16 +85,11 @@ class ceilometer::agent::notification (
$event_pipeline_publishers = ['gnocchi://'],
$manage_pipeline = false,
$pipeline_publishers = ['gnocchi://'],
$store_events = undef,
) {
include ::ceilometer::deps
include ::ceilometer::params
if $store_events != undef {
warning('store_events has been removed since Newton.')
}
ensure_resource('package', [$::ceilometer::params::agent_notification_package_name],
{
ensure => $package_ensure,
@ -157,7 +146,6 @@ class ceilometer::agent::notification (
ceilometer_config {
'notification/ack_on_event_error' : value => $ack_on_event_error;
'notification/store_events' : value => $store_events;
'notification/disable_non_metric_meters': value => $disable_non_metric_meters;
'notification/workers' : value => $notification_workers;
'notification/messaging_urls' : value => $messaging_urls, secret => true;

3
releasenotes/notes/remove_store_events-18a34502fe2387d1.yaml Normal file
View File

@ -0,0 +1,3 @@
---
other:
- removed deprecated ceilometer::agent::notification::store_events.