From 90f9f1dfa086f24975b4a11df9c7b46a07c430af Mon Sep 17 00:00:00 2001 From: Dean Troyer Date: Wed, 16 May 2012 11:25:34 -0500 Subject: Clean up tenant and server * remove '_' from class names * add class name to log instances * some cleanups to HACKING Change-Id: I1f6334318ee0d7d83cd3cea0e71ba4f05dd2b5c5 --- openstackclient/compute/v2/server.py | 21 ++++++++--------- openstackclient/identity/v2_0/tenant.py | 40 ++++++++++++++------------------- 2 files changed, 26 insertions(+), 35 deletions(-) (limited to 'openstackclient') diff --git a/openstackclient/compute/v2/server.py b/openstackclient/compute/v2/server.py index 201e29f7..ab7ef93e 100644 --- a/openstackclient/compute/v2/server.py +++ b/openstackclient/compute/v2/server.py @@ -44,14 +44,14 @@ def _format_servers_list_networks(server): return '; '.join(output) -class List_Server(command.OpenStackCommand, lister.Lister): - "List server command." +class ListServer(command.OpenStackCommand, lister.Lister): + """List server command""" api = 'compute' - log = logging.getLogger(__name__) + log = logging.getLogger(__name__ + '.ListServer') def get_parser(self, prog_name): - parser = super(List_Server, self).get_parser(prog_name) + parser = super(ListServer, self).get_parser(prog_name) parser.add_argument( '--reservation-id', help='only return instances that match the reservation', @@ -125,14 +125,14 @@ class List_Server(command.OpenStackCommand, lister.Lister): ) -class Show_Server(command.OpenStackCommand, show.ShowOne): - "Show server command." +class ShowServer(command.OpenStackCommand, show.ShowOne): + """Show server command""" api = 'compute' - log = logging.getLogger(__name__) + log = logging.getLogger(__name__ + '.ShowServer') def get_parser(self, prog_name): - parser = super(Show_Server, self).get_parser(prog_name) + parser = super(ShowServer, self).get_parser(prog_name) parser.add_argument( 'server', metavar='', @@ -164,7 +164,4 @@ class Show_Server(command.OpenStackCommand, show.ShowOne): # Remove a couple of values that are long and not too useful info.pop('links', None) - - columns = sorted(info.keys()) - values = [info[c] for c in columns] - return (columns, values) + return zip(*sorted(info.iteritems())) diff --git a/openstackclient/identity/v2_0/tenant.py b/openstackclient/identity/v2_0/tenant.py index 870b8454..0f400e9b 100644 --- a/openstackclient/identity/v2_0/tenant.py +++ b/openstackclient/identity/v2_0/tenant.py @@ -28,14 +28,14 @@ from openstackclient.common import command from openstackclient.common import utils -class Create_Tenant(command.OpenStackCommand, show.ShowOne): +class CreateTenant(command.OpenStackCommand, show.ShowOne): """Create tenant command""" api = 'identity' - log = logging.getLogger(__name__) + log = logging.getLogger(__name__ + '.CreateTenant') def get_parser(self, prog_name): - parser = super(Create_Tenant, self).get_parser(prog_name) + parser = super(CreateTenant, self).get_parser(prog_name) parser.add_argument( 'tenant_name', metavar='', @@ -73,20 +73,17 @@ class Create_Tenant(command.OpenStackCommand, show.ShowOne): info = {} info.update(tenant._info) + return zip(*sorted(info.iteritems())) - columns = sorted(info.keys()) - values = [info[c] for c in columns] - return (columns, values) - -class Delete_Tenant(command.OpenStackCommand): +class DeleteTenant(command.OpenStackCommand): """Delete tenant command""" api = 'identity' - log = logging.getLogger(__name__) + log = logging.getLogger(__name__ + '.DeleteTenant') def get_parser(self, prog_name): - parser = super(Delete_Tenant, self).get_parser(prog_name) + parser = super(DeleteTenant, self).get_parser(prog_name) parser.add_argument( 'tenant', metavar='', @@ -103,14 +100,14 @@ class Delete_Tenant(command.OpenStackCommand): return -class List_Tenant(command.OpenStackCommand, lister.Lister): +class ListTenant(command.OpenStackCommand, lister.Lister): """List tenant command""" api = 'identity' - log = logging.getLogger(__name__) + log = logging.getLogger(__name__ + '.ListTenant') def get_parser(self, prog_name): - parser = super(List_Tenant, self).get_parser(prog_name) + parser = super(ListTenant, self).get_parser(prog_name) parser.add_argument( '--long', action='store_true', @@ -134,14 +131,14 @@ class List_Tenant(command.OpenStackCommand, lister.Lister): ) -class Set_Tenant(command.OpenStackCommand): +class SetTenant(command.OpenStackCommand): """Set tenant command""" api = 'identity' - log = logging.getLogger(__name__) + log = logging.getLogger(__name__ + '.SetTenant') def get_parser(self, prog_name): - parser = super(Set_Tenant, self).get_parser(prog_name) + parser = super(SetTenant, self).get_parser(prog_name) parser.add_argument( 'tenant', metavar='', @@ -193,14 +190,14 @@ class Set_Tenant(command.OpenStackCommand): return -class Show_Tenant(command.OpenStackCommand, show.ShowOne): +class ShowTenant(command.OpenStackCommand, show.ShowOne): """Show tenant command""" api = 'identity' - log = logging.getLogger(__name__) + log = logging.getLogger(__name__ + '.ShowTenant') def get_parser(self, prog_name): - parser = super(Show_Tenant, self).get_parser(prog_name) + parser = super(ShowTenant, self).get_parser(prog_name) parser.add_argument( 'tenant', metavar='', @@ -216,7 +213,4 @@ class Show_Tenant(command.OpenStackCommand, show.ShowOne): info = {} info.update(tenant._info) - - columns = sorted(info.keys()) - values = [info[c] for c in columns] - return (columns, values) + return zip(*sorted(info.iteritems())) -- cgit v1.2.1