Set up logger of each command by metaclass

compute.v2.flavor is changed in this commit as an initial example.

Partial-Bug: #1532294
Change-Id: I262af6ade0ae03fbe1cd2ad198faf4ebb4ecf7ce

1 files changed, 6 insertions, 22 deletions

diff --git a/openstackclient/compute/v2/flavor.py b/openstackclient/compute/v2/flavor.py
index b34197e0..1af5fe70 100644
--- a/openstackclient/compute/v2/flavor.py
+++ b/openstackclient/compute/v2/flavor.py
@@ -15,22 +15,16 @@
 
 """Flavor 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 parseractions
 from openstackclient.common import utils
 
 
-class CreateFlavor(show.ShowOne):
+class CreateFlavor(command.ShowOne):
     """Create new flavor"""
 
-    log = logging.getLogger(__name__ + ".CreateFlavor")
-
     def get_parser(self, prog_name):
         parser = super(CreateFlavor, self).get_parser(prog_name)
         parser.add_argument(
@@ -128,8 +122,6 @@ class CreateFlavor(show.ShowOne):
 class DeleteFlavor(command.Command):
     """Delete flavor"""
 
-    log = logging.getLogger(__name__ + ".DeleteFlavor")
-
     def get_parser(self, prog_name):
         parser = super(DeleteFlavor, self).get_parser(prog_name)
         parser.add_argument(
@@ -147,11 +139,9 @@ class DeleteFlavor(command.Command):
         compute_client.flavors.delete(flavor.id)
 
 
-class ListFlavor(lister.Lister):
+class ListFlavor(command.Lister):
     """List flavors"""
 
-    log = logging.getLogger(__name__ + ".ListFlavor")
-
     def get_parser(self, prog_name):
         parser = super(ListFlavor, self).get_parser(prog_name)
         public_group = parser.add_mutually_exclusive_group()
@@ -231,11 +221,9 @@ class ListFlavor(lister.Lister):
                 ) for s in data))
 
 
-class ShowFlavor(show.ShowOne):
+class ShowFlavor(command.ShowOne):
     """Display flavor details"""
 
-    log = logging.getLogger(__name__ + ".ShowFlavor")
-
     def get_parser(self, prog_name):
         parser = super(ShowFlavor, self).get_parser(prog_name)
         parser.add_argument(
@@ -258,11 +246,9 @@ class ShowFlavor(show.ShowOne):
         return zip(*sorted(six.iteritems(flavor)))
 
 
-class SetFlavor(show.ShowOne):
+class SetFlavor(command.ShowOne):
     """Set flavor properties"""
 
-    log = logging.getLogger(__name__ + ".SetFlavor")
-
     def get_parser(self, prog_name):
         parser = super(SetFlavor, self).get_parser(prog_name)
         parser.add_argument(
@@ -292,11 +278,9 @@ class SetFlavor(show.ShowOne):
         return zip(*sorted(six.iteritems(flavor)))
 
 
-class UnsetFlavor(show.ShowOne):
+class UnsetFlavor(command.ShowOne):
     """Unset flavor properties"""
 
-    log = logging.getLogger(__name__ + ".UnsetFlavor")
-
     def get_parser(self, prog_name):
         parser = super(UnsetFlavor, self).get_parser(prog_name)
         parser.add_argument(