]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Set timetable for removal of oslo.messaging.notify.drivers
authorMatt Riedemann <mriedem@us.ibm.com>
Fri, 20 Nov 2015 02:43:07 +0000 (18:43 -0800)
committerMatt Riedemann <mriedem@us.ibm.com>
Sat, 5 Dec 2015 15:55:37 +0000 (07:55 -0800)
Icehouse is dead and gone, at least upstream. These special driver
registrations are not tested in the gate-tempest-dsvm-neutron-full job
which means they are also not tested in requirements constraints jobs.

oslo.messaging 2.6.0 broke these already by removing the internal modules,
which was fixed in o.m 3.0.0 with (deprecated) alias modules.

The minimum required version of o.m in mitaka is currently greater than
2.6.1, so we're OK to remove these once stable/mitaka is our oldest
supported branch. So add a TODO to remove these once liberty-eol happens.

Proper configuration for notification drivers happens through the
config file using the oslo_messaging options:

http://docs.openstack.org/developer/oslo.messaging/opts.html

Adds a release note for the deprecation and timetable.

Change-Id: I2c04117b0f531fe3604e74401d0ffd8d8d3dd308
Related-Bug: #1513630

releasenotes/notes/rm-notify-entry-points-aa442134a780469a.yaml [new file with mode: 0644]
setup.cfg

diff --git a/releasenotes/notes/rm-notify-entry-points-aa442134a780469a.yaml b/releasenotes/notes/rm-notify-entry-points-aa442134a780469a.yaml
new file mode 100644 (file)
index 0000000..880b868
--- /dev/null
@@ -0,0 +1,9 @@
+---
+prelude: |
+  oslo.messaging.notify.drivers entry points are deprecated
+other:
+  - |
+    The oslo.messaging.notify.drivers entry points that were left in tree for
+    backward compatibility with Icehouse are deprecated and will be removed
+    after liberty-eol. Configure notifications using the oslo_messaging
+    configuration options in neutron.conf.
index 133813de7ea128df9aed95a41fc5af6a63f77070..bf440154854352751feaec969adf9f2e528ed357 100644 (file)
--- a/setup.cfg
+++ b/setup.cfg
@@ -117,6 +117,7 @@ neutron.qos.agent_drivers =
 neutron.agent.linux.pd_drivers =
     dibbler = neutron.agent.linux.dibbler:PDDibbler
 # These are for backwards compat with Icehouse notification_driver configuration values
+# TODO(mriedem): Remove these once liberty-eol happens.
 oslo.messaging.notify.drivers =
     neutron.openstack.common.notifier.log_notifier = oslo_messaging.notify._impl_log:LogDriver
     neutron.openstack.common.notifier.no_op_notifier = oslo_messaging.notify._impl_noop:NoOpDriver