diff options
author | Julien Danjou <julien@danjou.info> | 2018-09-18 11:09:06 +0200 |
---|---|---|
committer | Julien Danjou <julien@danjou.info> | 2018-09-18 11:09:06 +0200 |
commit | 3d0b48133984d35a6a07c5e9963792faf961cc72 (patch) | |
tree | daa3f99da647bec61d1f4ff0e534bc36d92b4ab3 /ceilometer/event | |
parent | 42f6f68eb931b1089d46807f76693637f308a707 (diff) | |
download | ceilometer-3d0b48133984d35a6a07c5e9963792faf961cc72.tar.gz |
event: remove deprecated method
Change-Id: Iadda5e51d61c44c4dd11d7fc82bc7938f74baa5c
Diffstat (limited to 'ceilometer/event')
-rw-r--r-- | ceilometer/event/trait_plugins.py | 19 |
1 files changed, 6 insertions, 13 deletions
diff --git a/ceilometer/event/trait_plugins.py b/ceilometer/event/trait_plugins.py index 3dac1766..6961ea93 100644 --- a/ceilometer/event/trait_plugins.py +++ b/ceilometer/event/trait_plugins.py @@ -15,7 +15,6 @@ import abc -from debtcollector import moves from oslo_log import log from oslo_utils import timeutils import six @@ -49,10 +48,7 @@ class TraitPluginBase(object): """ super(TraitPluginBase, self).__init__() - @moves.moved_method('trait_values', version=6.0, removal_version="?") - def trait_value(self, match_list): - pass - + @abc.abstractmethod def trait_values(self, match_list): """Convert a set of fields to one or multiple Trait values. @@ -99,15 +95,12 @@ class TraitPluginBase(object): def __init__(self, **kw): super(DefaultPlugin, self).__init__() - def trait_value(self, match_list): + def trait_values(self, match_list): if not match_list: return None return [ match[1] for match in match_list] """ - # For backwards compatibility for the renamed method. - return [self.trait_value(match_list)] - class SplitterTraitPlugin(TraitPluginBase): """Plugin that splits a piece off of a string value.""" @@ -209,11 +202,11 @@ class TimedeltaPlugin(TraitPluginBase): # TODO(idegtiarov): refactor code to have meter_plugins separate from # trait_plugins - def trait_value(self, match_list): + def trait_values(self, match_list): if len(match_list) != 2: LOG.warning('Timedelta plugin is required two timestamp fields' ' to create timedelta value.') - return + return [None] start, end = match_list try: start_time = timeutils.parse_isotime(start[1]) @@ -224,5 +217,5 @@ class TimedeltaPlugin(TraitPluginBase): '%(err)s' % dict(start=start[0], end=end[0], err=err) ) - return - return abs((end_time - start_time).total_seconds()) + return [None] + return [abs((end_time - start_time).total_seconds())] |