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

8 files changed, 31 insertions, 161 deletions

diff --git a/openstackclient/identity/v2_0/catalog.py b/openstackclient/identity/v2_0/catalog.py
index 9bc2755a..c927943f 100644
--- a/openstackclient/identity/v2_0/catalog.py
+++ b/openstackclient/identity/v2_0/catalog.py
@@ -13,12 +13,9 @@
 
 """Identity v2 Service Catalog action implementations"""
 
-import logging
 import six
 
-from cliff import lister
-from cliff import show
-
+from openstackclient.common import command
 from openstackclient.common import utils
 from openstackclient.i18n import _  # noqa
 
@@ -37,12 +34,9 @@ def _format_endpoints(eps=None):
     return ret
 
 
-class ListCatalog(lister.Lister):
+class ListCatalog(command.Lister):
     """List services in the service catalog"""
 
-    log = logging.getLogger(__name__ + '.ListCatalog')
-
-    @utils.log_method(log)
     def take_action(self, parsed_args):
 
         # This is ugly because if auth hasn't happened yet we need
@@ -62,11 +56,9 @@ class ListCatalog(lister.Lister):
                 ) for s in data))
 
 
-class ShowCatalog(show.ShowOne):
+class ShowCatalog(command.ShowOne):
     """Display service catalog details"""
 
-    log = logging.getLogger(__name__ + '.ShowCatalog')
-
     def get_parser(self, prog_name):
         parser = super(ShowCatalog, self).get_parser(prog_name)
         parser.add_argument(
@@ -76,7 +68,6 @@ class ShowCatalog(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
 
         # This is ugly because if auth hasn't happened yet we need
diff --git a/openstackclient/identity/v2_0/ec2creds.py b/openstackclient/identity/v2_0/ec2creds.py
index a7730ce2..a16b3d9e 100644
--- a/openstackclient/identity/v2_0/ec2creds.py
+++ b/openstackclient/identity/v2_0/ec2creds.py
@@ -16,22 +16,16 @@
 
 """Identity v2 EC2 Credentials action implementations"""
 
-import logging
 import six
 
-from cliff import command
-from cliff import lister
-from cliff import show
-
+from openstackclient.common import command
 from openstackclient.common import utils
 from openstackclient.i18n import _  # noqa
 
 
-class CreateEC2Creds(show.ShowOne):
+class CreateEC2Creds(command.ShowOne):
     """Create EC2 credentials"""
 
-    log = logging.getLogger(__name__ + ".CreateEC2Creds")
-
     def get_parser(self, prog_name):
         parser = super(CreateEC2Creds, self).get_parser(prog_name)
         parser.add_argument(
@@ -52,7 +46,6 @@ class CreateEC2Creds(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -89,8 +82,6 @@ class CreateEC2Creds(show.ShowOne):
 class DeleteEC2Creds(command.Command):
     """Delete EC2 credentials"""
 
-    log = logging.getLogger(__name__ + '.DeleteEC2Creds')
-
     def get_parser(self, prog_name):
         parser = super(DeleteEC2Creds, self).get_parser(prog_name)
         parser.add_argument(
@@ -105,7 +96,6 @@ class DeleteEC2Creds(command.Command):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -121,11 +111,9 @@ class DeleteEC2Creds(command.Command):
         identity_client.ec2.delete(user, parsed_args.access_key)
 
 
-class ListEC2Creds(lister.Lister):
+class ListEC2Creds(command.Lister):
     """List EC2 credentials"""
 
-    log = logging.getLogger(__name__ + '.ListEC2Creds')
-
     def get_parser(self, prog_name):
         parser = super(ListEC2Creds, self).get_parser(prog_name)
         parser.add_argument(
@@ -135,7 +123,6 @@ class ListEC2Creds(lister.Lister):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -159,11 +146,9 @@ class ListEC2Creds(lister.Lister):
                 ) for s in data))
 
 
-class ShowEC2Creds(show.ShowOne):
+class ShowEC2Creds(command.ShowOne):
     """Display EC2 credentials details"""
 
-    log = logging.getLogger(__name__ + '.ShowEC2Creds')
-
     def get_parser(self, prog_name):
         parser = super(ShowEC2Creds, self).get_parser(prog_name)
         parser.add_argument(
@@ -178,7 +163,6 @@ class ShowEC2Creds(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
diff --git a/openstackclient/identity/v2_0/endpoint.py b/openstackclient/identity/v2_0/endpoint.py
index 1744cc72..08c09565 100644
--- a/openstackclient/identity/v2_0/endpoint.py
+++ b/openstackclient/identity/v2_0/endpoint.py
@@ -15,23 +15,17 @@
 
 """Endpoint action implementations"""
 
-import logging
 import six
 
-from cliff import command
-from cliff import lister
-from cliff import show
-
+from openstackclient.common import command
 from openstackclient.common import utils
 from openstackclient.i18n import _  # noqa
 from openstackclient.identity import common
 
 
-class CreateEndpoint(show.ShowOne):
+class CreateEndpoint(command.ShowOne):
     """Create new endpoint"""
 
-    log = logging.getLogger(__name__ + '.CreateEndpoint')
-
     def get_parser(self, prog_name):
         parser = super(CreateEndpoint, self).get_parser(prog_name)
         parser.add_argument(
@@ -62,7 +56,6 @@ class CreateEndpoint(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         service = common.find_service(identity_client, parsed_args.service)
@@ -83,8 +76,6 @@ class CreateEndpoint(show.ShowOne):
 class DeleteEndpoint(command.Command):
     """Delete endpoint"""
 
-    log = logging.getLogger(__name__ + '.DeleteEndpoint')
-
     def get_parser(self, prog_name):
         parser = super(DeleteEndpoint, self).get_parser(prog_name)
         parser.add_argument(
@@ -93,18 +84,15 @@ class DeleteEndpoint(command.Command):
             help=_('Endpoint ID to delete'))
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         identity_client.endpoints.delete(parsed_args.endpoint)
         return
 
 
-class ListEndpoint(lister.Lister):
+class ListEndpoint(command.Lister):
     """List endpoints"""
 
-    log = logging.getLogger(__name__ + '.ListEndpoint')
-
     def get_parser(self, prog_name):
         parser = super(ListEndpoint, self).get_parser(prog_name)
         parser.add_argument(
@@ -115,7 +103,6 @@ class ListEndpoint(lister.Lister):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         if parsed_args.long:
@@ -136,11 +123,9 @@ class ListEndpoint(lister.Lister):
                 ) for s in data))
 
 
-class ShowEndpoint(show.ShowOne):
+class ShowEndpoint(command.ShowOne):
     """Display endpoint details"""
 
-    log = logging.getLogger(__name__ + '.ShowEndpoint')
-
     def get_parser(self, prog_name):
         parser = super(ShowEndpoint, self).get_parser(prog_name)
         parser.add_argument(
@@ -150,7 +135,6 @@ class ShowEndpoint(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         data = identity_client.endpoints.list()
diff --git a/openstackclient/identity/v2_0/project.py b/openstackclient/identity/v2_0/project.py
index 4330c79c..9e26c308 100644
--- a/openstackclient/identity/v2_0/project.py
+++ b/openstackclient/identity/v2_0/project.py
@@ -15,24 +15,19 @@
 
 """Identity v2 Project action implementations"""
 
-import logging
 import six
 
-from cliff import command
-from cliff import lister
-from cliff import show
 from keystoneauth1 import exceptions as ks_exc
 
+from openstackclient.common import command
 from openstackclient.common import parseractions
 from openstackclient.common import utils
 from openstackclient.i18n import _  # noqa
 
 
-class CreateProject(show.ShowOne):
+class CreateProject(command.ShowOne):
     """Create new project"""
 
-    log = logging.getLogger(__name__ + '.CreateProject')
-
     def get_parser(self, prog_name):
         parser = super(CreateProject, self).get_parser(prog_name)
         parser.add_argument(
@@ -70,7 +65,6 @@ class CreateProject(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -106,8 +100,6 @@ class CreateProject(show.ShowOne):
 class DeleteProject(command.Command):
     """Delete project(s)"""
 
-    log = logging.getLogger(__name__ + '.DeleteProject')
-
     def get_parser(self, prog_name):
         parser = super(DeleteProject, self).get_parser(prog_name)
         parser.add_argument(
@@ -118,7 +110,6 @@ class DeleteProject(command.Command):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -131,11 +122,9 @@ class DeleteProject(command.Command):
         return
 
 
-class ListProject(lister.Lister):
+class ListProject(command.Lister):
     """List projects"""
 
-    log = logging.getLogger(__name__ + '.ListProject')
-
     def get_parser(self, prog_name):
         parser = super(ListProject, self).get_parser(prog_name)
         parser.add_argument(
@@ -146,7 +135,6 @@ class ListProject(lister.Lister):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         if parsed_args.long:
             columns = ('ID', 'Name', 'Description', 'Enabled')
@@ -163,8 +151,6 @@ class ListProject(lister.Lister):
 class SetProject(command.Command):
     """Set project properties"""
 
-    log = logging.getLogger(__name__ + '.SetProject')
-
     def get_parser(self, prog_name):
         parser = super(SetProject, self).get_parser(prog_name)
         parser.add_argument(
@@ -202,7 +188,6 @@ class SetProject(command.Command):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -240,11 +225,9 @@ class SetProject(command.Command):
         return
 
 
-class ShowProject(show.ShowOne):
+class ShowProject(command.ShowOne):
     """Display project details"""
 
-    log = logging.getLogger(__name__ + '.ShowProject')
-
     def get_parser(self, prog_name):
         parser = super(ShowProject, self).get_parser(prog_name)
         parser.add_argument(
@@ -253,7 +236,6 @@ class ShowProject(show.ShowOne):
             help=_('Project to display (name or ID)'))
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -303,8 +285,6 @@ class ShowProject(show.ShowOne):
 class UnsetProject(command.Command):
     """Unset project properties"""
 
-    log = logging.getLogger(__name__ + '.UnsetProject')
-
     def get_parser(self, prog_name):
         parser = super(UnsetProject, self).get_parser(prog_name)
         parser.add_argument(
@@ -323,7 +303,6 @@ class UnsetProject(command.Command):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         project = utils.find_resource(
diff --git a/openstackclient/identity/v2_0/role.py b/openstackclient/identity/v2_0/role.py
index cab6b4a5..892ce006 100644
--- a/openstackclient/identity/v2_0/role.py
+++ b/openstackclient/identity/v2_0/role.py
@@ -15,24 +15,19 @@
 
 """Identity v2 Role action implementations"""
 
-import logging
 import six
 
-from cliff import command
-from cliff import lister
-from cliff import show
 from keystoneauth1 import exceptions as ks_exc
 
+from openstackclient.common import command
 from openstackclient.common import exceptions
 from openstackclient.common import utils
 from openstackclient.i18n import _  # noqa
 
 
-class AddRole(show.ShowOne):
+class AddRole(command.ShowOne):
     """Add role to project:user"""
 
-    log = logging.getLogger(__name__ + '.AddRole')
-
     def get_parser(self, prog_name):
         parser = super(AddRole, self).get_parser(prog_name)
         parser.add_argument(
@@ -54,7 +49,6 @@ class AddRole(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         role = utils.find_resource(identity_client.roles, parsed_args.role)
@@ -74,11 +68,9 @@ class AddRole(show.ShowOne):
         return zip(*sorted(six.iteritems(info)))
 
 
-class CreateRole(show.ShowOne):
+class CreateRole(command.ShowOne):
     """Create new role"""
 
-    log = logging.getLogger(__name__ + '.CreateRole')
-
     def get_parser(self, prog_name):
         parser = super(CreateRole, self).get_parser(prog_name)
         parser.add_argument(
@@ -93,7 +85,6 @@ class CreateRole(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         try:
@@ -116,8 +107,6 @@ class CreateRole(show.ShowOne):
 class DeleteRole(command.Command):
     """Delete role(s)"""
 
-    log = logging.getLogger(__name__ + '.DeleteRole')
-
     def get_parser(self, prog_name):
         parser = super(DeleteRole, self).get_parser(prog_name)
         parser.add_argument(
@@ -128,7 +117,6 @@ class DeleteRole(command.Command):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -141,11 +129,9 @@ class DeleteRole(command.Command):
         return
 
 
-class ListRole(lister.Lister):
+class ListRole(command.Lister):
     """List roles"""
 
-    log = logging.getLogger(__name__ + '.ListRole')
-
     def get_parser(self, prog_name):
         parser = super(ListRole, self).get_parser(prog_name)
         parser.add_argument(
@@ -160,7 +146,6 @@ class ListRole(lister.Lister):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         auth_ref = self.app.client_manager.auth_ref
@@ -222,11 +207,9 @@ class ListRole(lister.Lister):
                 ) for s in data))
 
 
-class ListUserRole(lister.Lister):
+class ListUserRole(command.Lister):
     """List user-role assignments"""
 
-    log = logging.getLogger(__name__ + '.ListUserRole')
-
     def get_parser(self, prog_name):
         parser = super(ListUserRole, self).get_parser(prog_name)
         parser.add_argument(
@@ -242,7 +225,6 @@ class ListUserRole(lister.Lister):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         auth_ref = self.app.client_manager.auth_ref
@@ -293,8 +275,6 @@ class ListUserRole(lister.Lister):
 class RemoveRole(command.Command):
     """Remove role from project : user"""
 
-    log = logging.getLogger(__name__ + '.RemoveRole')
-
     def get_parser(self, prog_name):
         parser = super(RemoveRole, self).get_parser(prog_name)
         parser.add_argument(
@@ -316,7 +296,6 @@ class RemoveRole(command.Command):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         role = utils.find_resource(identity_client.roles, parsed_args.role)
@@ -331,11 +310,9 @@ class RemoveRole(command.Command):
             project.id)
 
 
-class ShowRole(show.ShowOne):
+class ShowRole(command.ShowOne):
     """Display role details"""
 
-    log = logging.getLogger(__name__ + '.ShowRole')
-
     def get_parser(self, prog_name):
         parser = super(ShowRole, self).get_parser(prog_name)
         parser.add_argument(
@@ -345,7 +322,6 @@ class ShowRole(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         role = utils.find_resource(identity_client.roles, parsed_args.role)
diff --git a/openstackclient/identity/v2_0/service.py b/openstackclient/identity/v2_0/service.py
index c9d48441..3af85d7d 100644
--- a/openstackclient/identity/v2_0/service.py
+++ b/openstackclient/identity/v2_0/service.py
@@ -16,24 +16,18 @@
 """Service action implementations"""
 
 import argparse
-import logging
 import six
 
-from cliff import command
-from cliff import lister
-from cliff import show
-
+from openstackclient.common import command
 from openstackclient.common import exceptions
 from openstackclient.common import utils
 from openstackclient.i18n import _  # noqa
 from openstackclient.identity import common
 
 
-class CreateService(show.ShowOne):
+class CreateService(command.ShowOne):
     """Create new service"""
 
-    log = logging.getLogger(__name__ + '.CreateService')
-
     def get_parser(self, prog_name):
         parser = super(CreateService, self).get_parser(prog_name)
         parser.add_argument(
@@ -59,7 +53,6 @@ class CreateService(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -95,8 +88,6 @@ class CreateService(show.ShowOne):
 class DeleteService(command.Command):
     """Delete service"""
 
-    log = logging.getLogger(__name__ + '.DeleteService')
-
     def get_parser(self, prog_name):
         parser = super(DeleteService, self).get_parser(prog_name)
         parser.add_argument(
@@ -106,7 +97,6 @@ class DeleteService(command.Command):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         service = common.find_service(identity_client, parsed_args.service)
@@ -114,11 +104,9 @@ class DeleteService(command.Command):
         return
 
 
-class ListService(lister.Lister):
+class ListService(command.Lister):
     """List services"""
 
-    log = logging.getLogger(__name__ + '.ListService')
-
     def get_parser(self, prog_name):
         parser = super(ListService, self).get_parser(prog_name)
         parser.add_argument(
@@ -129,7 +117,6 @@ class ListService(lister.Lister):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
 
         if parsed_args.long:
@@ -143,11 +130,9 @@ class ListService(lister.Lister):
         )
 
 
-class ShowService(show.ShowOne):
+class ShowService(command.ShowOne):
     """Display service details"""
 
-    log = logging.getLogger(__name__ + '.ShowService')
-
     def get_parser(self, prog_name):
         parser = super(ShowService, self).get_parser(prog_name)
         parser.add_argument(
@@ -163,7 +148,6 @@ class ShowService(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
         auth_ref = self.app.client_manager.auth_ref
diff --git a/openstackclient/identity/v2_0/token.py b/openstackclient/identity/v2_0/token.py
index 5fed58e5..db38fae8 100644
--- a/openstackclient/identity/v2_0/token.py
+++ b/openstackclient/identity/v2_0/token.py
@@ -15,26 +15,19 @@
 
 """Identity v2 Token action implementations"""
 
-import logging
 import six
 
-from cliff import command
-from cliff import show
-
-from openstackclient.common import utils
+from openstackclient.common import command
 from openstackclient.i18n import _  # noqa
 
 
-class IssueToken(show.ShowOne):
+class IssueToken(command.ShowOne):
     """Issue new token"""
 
-    log = logging.getLogger(__name__ + '.IssueToken')
-
     def get_parser(self, prog_name):
         parser = super(IssueToken, self).get_parser(prog_name)
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
 
         token = self.app.client_manager.auth_ref.service_catalog.get_token()
@@ -45,8 +38,6 @@ class IssueToken(show.ShowOne):
 class RevokeToken(command.Command):
     """Revoke existing token"""
 
-    log = logging.getLogger(__name__ + '.RevokeToken')
-
     def get_parser(self, prog_name):
         parser = super(RevokeToken, self).get_parser(prog_name)
         parser.add_argument(
@@ -57,7 +48,6 @@ class RevokeToken(command.Command):
         return parser
 
     def take_action(self, parsed_args):
-        self.log.debug('take_action(%s)' % parsed_args)
         identity_client = self.app.client_manager.identity
 
         identity_client.tokens.delete(parsed_args.token)
diff --git a/openstackclient/identity/v2_0/user.py b/openstackclient/identity/v2_0/user.py
index e2b285bd..3d848737 100644
--- a/openstackclient/identity/v2_0/user.py
+++ b/openstackclient/identity/v2_0/user.py
@@ -15,23 +15,18 @@
 
 """Identity v2.0 User action implementations"""
 
-import logging
 import six
 
-from cliff import command
-from cliff import lister
-from cliff import show
 from keystoneauth1 import exceptions as ks_exc
 
+from openstackclient.common import command
 from openstackclient.common import utils
 from openstackclient.i18n import _  # noqa
 
 
-class CreateUser(show.ShowOne):
+class CreateUser(command.ShowOne):
     """Create new user"""
 
-    log = logging.getLogger(__name__ + '.CreateUser')
-
     def get_parser(self, prog_name):
         parser = super(CreateUser, self).get_parser(prog_name)
         parser.add_argument(
@@ -78,7 +73,6 @@ class CreateUser(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -130,8 +124,6 @@ class CreateUser(show.ShowOne):
 class DeleteUser(command.Command):
     """Delete user(s)"""
 
-    log = logging.getLogger(__name__ + '.DeleteUser')
-
     def get_parser(self, prog_name):
         parser = super(DeleteUser, self).get_parser(prog_name)
         parser.add_argument(
@@ -142,7 +134,6 @@ class DeleteUser(command.Command):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -155,11 +146,9 @@ class DeleteUser(command.Command):
         return
 
 
-class ListUser(lister.Lister):
+class ListUser(command.Lister):
     """List users"""
 
-    log = logging.getLogger(__name__ + '.ListUser')
-
     def get_parser(self, prog_name):
         parser = super(ListUser, self).get_parser(prog_name)
         parser.add_argument(
@@ -174,7 +163,6 @@ class ListUser(lister.Lister):
             help=_('List additional fields in output'))
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -248,8 +236,6 @@ class ListUser(lister.Lister):
 class SetUser(command.Command):
     """Set user properties"""
 
-    log = logging.getLogger(__name__ + '.SetUser')
-
     def get_parser(self, prog_name):
         parser = super(SetUser, self).get_parser(prog_name)
         parser.add_argument(
@@ -296,7 +282,6 @@ class SetUser(command.Command):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity
 
@@ -348,11 +333,9 @@ class SetUser(command.Command):
         return
 
 
-class ShowUser(show.ShowOne):
+class ShowUser(command.ShowOne):
     """Display user details"""
 
-    log = logging.getLogger(__name__ + '.ShowUser')
-
     def get_parser(self, prog_name):
         parser = super(ShowUser, self).get_parser(prog_name)
         parser.add_argument(
@@ -362,7 +345,6 @@ class ShowUser(show.ShowOne):
         )
         return parser
 
-    @utils.log_method(log)
     def take_action(self, parsed_args):
         identity_client = self.app.client_manager.identity