log take_action parameters in a single place

Previously each command logs take_action parameters explicitly
by using @utils.log_method decorator or log.debug().
Some commands have no logging.

This commit calls a logger in the base class and
drops all logging definition from individual commands.

Closes-Bug: #1532294
Change-Id: I43cd0290a4353c68c075bade9571c940733da1be

1 files changed, 3 insertions, 12 deletions

diff --git a/openstackclient/compute/v2/usage.py b/openstackclient/compute/v2/usage.py
index 4e7cf100..baa50170 100644
--- a/openstackclient/compute/v2/usage.py
+++ b/openstackclient/compute/v2/usage.py
@@ -16,21 +16,17 @@
 """Usage action implementations"""
 
 import datetime
-import logging
 import sys
 
-from cliff import lister
-from cliff import show
 import six
 
+from openstackclient.common import command
 from openstackclient.common import utils
 
 
-class ListUsage(lister.Lister):
+class ListUsage(command.Lister):
     """List resource usage per project"""
 
-    log = logging.getLogger(__name__ + ".ListUsage")
-
     def get_parser(self, prog_name):
         parser = super(ListUsage, self).get_parser(prog_name)
         parser.add_argument(
@@ -49,7 +45,6 @@ class ListUsage(lister.Lister):
         return parser
 
     def take_action(self, parsed_args):
-        self.log.debug("take_action(%s)", parsed_args)
 
         def _format_project(project):
             if not project:
@@ -118,11 +113,9 @@ class ListUsage(lister.Lister):
                 ) for s in usage_list))
 
 
-class ShowUsage(show.ShowOne):
+class ShowUsage(command.ShowOne):
     """Show resource usage for a single project"""
 
-    log = logging.getLogger(__name__ + ".ShowUsage")
-
     def get_parser(self, prog_name):
         parser = super(ShowUsage, self).get_parser(prog_name)
         parser.add_argument(
@@ -147,8 +140,6 @@ class ShowUsage(show.ShowOne):
         return parser
 
     def take_action(self, parsed_args):
-        self.log.debug("take_action(%s)", parsed_args)
-
         identity_client = self.app.client_manager.identity
         compute_client = self.app.client_manager.compute
         dateformat = "%Y-%m-%d"