openstack/python-searchlightclient
Code Issues Proposed changes

Remove duplicated take_action logging

Browse Source 
This commit drops utils.log_method decorators.
As a result of logging refactoring in openstackclient [1],
utils.log_method decorator is no longer used in openstackclient
and we are planning to drop utils.log_method.

searchlightclient is the only project that uses log_method
decorator. take_action logging is duplicated, so we can safely
drop log_method decorator.

Once this patch is merged, openstackclient can drop utils.log_method
because there is no released searchlightclient.

[1] https://review.openstack.org/#/q/topic:bug/1532294

Change-Id: Ib94e7ba77262a9a8cbfce71f3083c47cb1973364
Related-Bug: #1532294
This commit is contained in:
Akihiro Motoki
2016-02-02 11:17:29 +09:00
parent 19c5e37246
commit ce396c80c7
3 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
searchlightclient/osc/v1/facet.py
View File

@@ -46,7 +46,6 @@ class ListFacet(lister.Lister):
) )
return parser return parser
@utils.log_method(log)
def take_action(self, parsed_args): def take_action(self, parsed_args):
self.log.debug("take_action(%s)", parsed_args) self.log.debug("take_action(%s)", parsed_args)

1
searchlightclient/osc/v1/resource_type.py
View File

@@ -24,7 +24,6 @@ class ListResourceType(lister.Lister):
log = logging.getLogger(__name__ + ".ListResourceType") log = logging.getLogger(__name__ + ".ListResourceType")
@utils.log_method(log)
def take_action(self, parsed_args): def take_action(self, parsed_args):
self.log.debug("take_action(%s)", parsed_args) self.log.debug("take_action(%s)", parsed_args)

1
searchlightclient/osc/v1/search.py
View File

@@ -65,7 +65,6 @@ class SearchResource(lister.Lister):
) )
return parser return parser
@utils.log_method(log)
def take_action(self, parsed_args): def take_action(self, parsed_args):
self.log.debug("take_action(%s)", parsed_args) self.log.debug("take_action(%s)", parsed_args)