diff --git a/cloudkitty/collector/gnocchi.py b/cloudkitty/collector/gnocchi.py index e02067f3..af0ddc05 100644 --- a/cloudkitty/collector/gnocchi.py +++ b/cloudkitty/collector/gnocchi.py @@ -385,6 +385,7 @@ class GnocchiCollector(collector.BaseCollector): 'resource_type': resource_type, 'start': start, 'stop': end, + 'use_history': True, 'groupby': self.conf[metric_name]['groupby'], 'search': self.extend_filter(*query_parameters), } diff --git a/cloudkitty/tests/collectors/test_gnocchi.py b/cloudkitty/tests/collectors/test_gnocchi.py index f765ed09..26b0318d 100644 --- a/cloudkitty/tests/collectors/test_gnocchi.py +++ b/cloudkitty/tests/collectors/test_gnocchi.py @@ -211,7 +211,8 @@ class GnocchiCollectorAggregationOperationTest(tests.TestCase): resource_type='resource_x', search={'=': {'type': 'resource_x'}}, start=self.start, stop=self.end, - granularity=3600 + granularity=3600, + use_history=True ) def test_multiple_confs(self): diff --git a/releasenotes/notes/add_warning_regarding_gnocchi_version-99d5213c35950e39.yaml b/releasenotes/notes/add_warning_regarding_gnocchi_version-99d5213c35950e39.yaml new file mode 100644 index 00000000..d91a9a1d --- /dev/null +++ b/releasenotes/notes/add_warning_regarding_gnocchi_version-99d5213c35950e39.yaml @@ -0,0 +1,14 @@ +--- +fixes: + - | + CloudKitty will always use the correct metadata for the + processing and reprocessing jobs. This means, we always use + the metadata for the timestamp that we are collecting at + Gnocchi backend.This is achieved with the use of + ``use_history=true`` in Gnocchi, which was released under + `version 4.5.0 `__. + Before that release, the ``aggregates`` API would only return + the latest metadata for the resource of the metric being handled. + Therefore, for CloudKitty processing and reprocessing, we would + always have the possibility of using the wrong attribute version + to rate the computing resources. \ No newline at end of file