From da7dbc4593269eab36186fdefca29588ac306ca9 Mon Sep 17 00:00:00 2001 From: Luka Peschke Date: Wed, 24 Jan 2018 09:56:06 +0100 Subject: [PATCH] Deprecate the ceilometer collector The Ceilometer API has been deprecated for a while now. Some of the meter types queried by the collector (instance, image) have been removed from the Telemetry project in the Ocata release. In order to make sure that the warning will be seen, it has been put in the constructor of the Worker class. We should completely remove the collector during the next release cycle. Change-Id: Ieaa2bc585d0cbcf7b035da627e189c920c9a8e08 --- cloudkitty/orchestrator.py | 4 ++++ .../deprecate-ceilometer-collector-6d8f72c84b95662b.yaml | 5 +++++ 2 files changed, 9 insertions(+) create mode 100644 releasenotes/notes/deprecate-ceilometer-collector-6d8f72c84b95662b.yaml diff --git a/cloudkitty/orchestrator.py b/cloudkitty/orchestrator.py index 2a5614b7..14a44b4d 100644 --- a/cloudkitty/orchestrator.py +++ b/cloudkitty/orchestrator.py @@ -153,6 +153,10 @@ class APIWorker(BaseWorker): class Worker(BaseWorker): def __init__(self, collector, storage, tenant_id=None): self._collector = collector + if getattr(self._collector, 'collector_name') == 'ceilometer': + LOG.warning('Ceilometer collector is deprecated and will be ' + 'removed during R cycle. This collector only works ' + 'with versions prior to Ocata') self._storage = storage self._period = METRICS_CONF['period'] self._wait_time = METRICS_CONF['wait_periods'] * self._period diff --git a/releasenotes/notes/deprecate-ceilometer-collector-6d8f72c84b95662b.yaml b/releasenotes/notes/deprecate-ceilometer-collector-6d8f72c84b95662b.yaml new file mode 100644 index 00000000..fbce42d5 --- /dev/null +++ b/releasenotes/notes/deprecate-ceilometer-collector-6d8f72c84b95662b.yaml @@ -0,0 +1,5 @@ +--- +deprecations: + - | + The ceilometer collector has been deprecated. Gnocchi should be used by + default. The collector will be removed during the Rocky development cycle.