openstack/deb-gnocchi
Code Issues Proposed changes

drop _delete_unaggregated_timeserie

Browse Source 
it's not used anywhere:
- file driver just deletes entire folder
- ceph driver just deletes as part of _delete_metric as unaggregated
object is what contains omaps references to aggregate objects

Change-Id: I83dfe73e8256dfa12df346b9615b3f2874e96773
This commit is contained in:
gord chung 2017-04-13 15:46:38 +00:00
parent baa29e863d
commit e537e7caf8
2 changed files with 0 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gnocchi/storage/ceph.py
View File

@ -157,10 +157,6 @@ class CephStorage(_carbonara.CarbonaraBasedStorage):
self.ioctx.write_full( self.ioctx.write_full(
self._build_unaggregated_timeserie_path(metric, version), data) self._build_unaggregated_timeserie_path(metric, version), data)
def _delete_unaggregated_timeserie(self, metric, version=3):
self.ioctx.aio_remove(
self._build_unaggregated_timeserie_path(metric, version))
def _get_object_content(self, name): def _get_object_content(self, name):
offset = 0 offset = 0
content = b'' content = b''

9
gnocchi/storage/file.py
View File

@ -98,15 +98,6 @@ class FileStorage(_carbonara.CarbonaraBasedStorage):
raise storage.MetricDoesNotExist(metric) raise storage.MetricDoesNotExist(metric)
raise raise
def _delete_unaggregated_timeserie(self, metric, version=3):
path = self._build_unaggregated_timeserie_path(metric, version)
try:
os.unlink(path)
except IOError as e:
if e.errno == errno.ENOENT:
raise storage.MetricDoesNotExist(metric)
raise
def _list_split_keys_for_metric(self, metric, aggregation, granularity, def _list_split_keys_for_metric(self, metric, aggregation, granularity,
version=3): version=3):
try: try: