openstack
/
kolla-ansible
Code Issues Proposed changes

Remove enable_cadf_notifications variable 

The variable enable_cadf_notifications is deprecated and marked
for removal during the U cycle.

Change-Id: I5e4d20d112db2392b55a0788f4d704ab6ca6112f
This commit is contained in:
Christian Berendt 2020-03-03 19:52:21 +01:00
parent 2978f13714
commit 77a1d27938
4 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ansible/group_vars/all.yml
View File

@ -542,8 +542,6 @@ enable_haproxy_memcached: "no"
enable_aodh: "no"
enable_barbican: "no"
enable_blazar: "no"
# NOTE: This variable has been deprecated and will be removed in the U cycle.
enable_cadf_notifications: "no"
enable_ceilometer: "no"
enable_ceilometer_ipmi: "no"
enable_cells: "no"

2
ansible/roles/keystone/defaults/main.yml
View File

@ -118,7 +118,7 @@ keystone_source_version: "{{ kolla_source_version }}"
####################
# Notifications
####################
keystone_default_notifications_topic_enabled: "{{ (enable_ceilometer | bool ) or (enable_cadf_notifications | bool)}}"
keystone_default_notifications_topic_enabled: "{{ enable_ceilometer | bool }}"
keystone_default_notifications_topic_name: "notifications"
keystone_notification_topics:

2
etc/kolla/globals.yml
View File

@ -226,8 +226,6 @@
#enable_aodh: "no"
#enable_barbican: "no"
#enable_blazar: "no"
# NOTE: This variable has been deprecated and will be removed in the U cycle.
#enable_cadf_notifications: "no"
#enable_ceilometer: "no"
#enable_ceilometer_ipmi: "no"
#enable_cells: "no"

7
releasenotes/notes/remove-enable-cadf-notifications-variable-5af76a972725f202.yaml Normal file
View File

@ -0,0 +1,7 @@
---
upgrade:
- |
The ``enable_cadf_notifications`` variable was removed. CADF is the
default notification format in keystone. To enable keystone notifications,
users can now set ``keystone_default_notifications_topic_enabled`` to
``yes`` or enable Ceilometer via ``enable_ceilometer``.