Remove tenant round 2 - Identity API

Rename everything in the Identity v2 code including the file and classes.
* role, tenant, user

Change-Id: I0b99c60a24f5875120136d2f216aa3b6bce1c641

Author: Dean Troyer

openstackclient/identity/v2_0/tenant.py openstackclient/identity/v2_0/project.pyopenstackclient/identity/v2_0/tenant.py renamed to openstackclient/identity/v2_0/project.py

@@ -13,11 +13,10 @@
 #   under the License.
 #
 
-"""Tenant action implementations"""
+"""Identity v2 Project action implementations"""
 
 import logging
 import six
-import sys
 
 from cliff import command
 from cliff import lister
@@ -26,82 +25,90 @@
 from openstackclient.common import utils
 
 
-class CreateTenant(show.ShowOne):
-    """Create tenant command"""
+class CreateProject(show.ShowOne):
+    """Create new project"""
 
-    log = logging.getLogger(__name__ + '.CreateTenant')
+    log = logging.getLogger(__name__ + '.CreateProject')
 
     def get_parser(self, prog_name):
-        parser = super(CreateTenant, self).get_parser(prog_name)
+        parser = super(CreateProject, self).get_parser(prog_name)
         parser.add_argument(
-            'tenant_name',
-            metavar='<tenant-name>',
-            help='New tenant name')
+            'project_name',
+            metavar='<project-name>',
+            help='New project name',
+        )
         parser.add_argument(
             '--description',
-            metavar='<tenant-description>',
-            help='New tenant description')
+            metavar='<project-description>',
+            help='New project description',
+        )
         enable_group = parser.add_mutually_exclusive_group()
         enable_group.add_argument(
             '--enable',
             dest='enabled',
             action='store_true',
             default=True,
-            help='Enable tenant')
+            help='Enable project',
+        )
         enable_group.add_argument(
             '--disable',
             dest='enabled',
             action='store_false',
-            help='Disable tenant')
+            help='Disable project',
+        )
         return parser
 
     def take_action(self, parsed_args):
         self.log.debug('take_action(%s)' % parsed_args)
         identity_client = self.app.client_manager.identity
-        tenant = identity_client.tenants.create(
-            parsed_args.tenant_name,
+        project = identity_client.tenants.create(
+            parsed_args.project_name,
             description=parsed_args.description,
             enabled=parsed_args.enabled)
 
         info = {}
-        info.update(tenant._info)
+        info.update(project._info)
         return zip(*sorted(six.iteritems(info)))
 
 
-class DeleteTenant(command.Command):
-    """Delete tenant command"""
+class DeleteProject(command.Command):
+    """Delete project"""
 
-    log = logging.getLogger(__name__ + '.DeleteTenant')
+    log = logging.getLogger(__name__ + '.DeleteProject')
 
     def get_parser(self, prog_name):
-        parser = super(DeleteTenant, self).get_parser(prog_name)
+        parser = super(DeleteProject, self).get_parser(prog_name)
         parser.add_argument(
-            'tenant',
-            metavar='<tenant>',
-            help='Name or ID of tenant to delete')
+            'project',
+            metavar='<project>',
+            help='Project to delete (name or ID)',
+        )
         return parser
 
     def take_action(self, parsed_args):
         self.log.debug('take_action(%s)' % parsed_args)
         identity_client = self.app.client_manager.identity
-        tenant = utils.find_resource(identity_client.tenants,
-                                     parsed_args.tenant)
-        identity_client.tenants.delete(tenant.id)
+        project = utils.find_resource(
+            identity_client.tenants,
+            parsed_args.project,
+        )
+        identity_client.tenants.delete(project.id)
         return
 
 
-class ListTenant(lister.Lister):
-    """List tenant command"""
+class ListProject(lister.Lister):
+    """List projects"""
 
-    log = logging.getLogger(__name__ + '.ListTenant')
+    log = logging.getLogger(__name__ + '.ListProject')
 
     def get_parser(self, prog_name):
-        parser = super(ListTenant, self).get_parser(prog_name)
+        parser = super(ListProject, self).get_parser(prog_name)
         parser.add_argument(
             '--long',
             action='store_true',
             default=False,
-            help='Additional fields are listed in output')
+            help='List additional fields in output',
+        )
         return parser
 
     def take_action(self, parsed_args):
@@ -118,44 +125,51 @@ def take_action(self, parsed_args):
                 ) for s in data))
 
 
-class SetTenant(command.Command):
-    """Set tenant command"""
+class SetProject(command.Command):
+    """Set project properties"""
 
-    log = logging.getLogger(__name__ + '.SetTenant')
+    log = logging.getLogger(__name__ + '.SetProject')
 
     def get_parser(self, prog_name):
-        parser = super(SetTenant, self).get_parser(prog_name)
+        parser = super(SetProject, self).get_parser(prog_name)
         parser.add_argument(
-            'tenant',
-            metavar='<tenant>',
-            help='Name or ID of tenant to change')
+            'project',
+            metavar='<project>',
+            help='Project to change (name or ID)',
+        )
         parser.add_argument(
             '--name',
-            metavar='<new-tenant-name>',
-            help='New tenant name')
+            metavar='<new-project-name>',
+            help='New project name',
+        )
         parser.add_argument(
             '--description',
-            metavar='<tenant-description>',
-            help='New tenant description')
+            metavar='<project-description>',
+            help='New project description',
+        )
         enable_group = parser.add_mutually_exclusive_group()
         enable_group.add_argument(
             '--enable',
             dest='enabled',
             action='store_true',
             default=True,
-            help='Enable tenant (default)')
+            help='Enable project (default)',
+        )
         enable_group.add_argument(
             '--disable',
             dest='enabled',
             action='store_false',
-            help='Disable tenant')
+            help='Disable project',
+        )
         return parser
 
     def take_action(self, parsed_args):
         self.log.debug('take_action(%s)' % parsed_args)
         identity_client = self.app.client_manager.identity
-        tenant = utils.find_resource(identity_client.tenants,
-                                     parsed_args.tenant)
+        project = utils.find_resource(
+            identity_client.tenants,
+            parsed_args.project,
+        )
         kwargs = {}
         if parsed_args.name:
             kwargs['name'] = parsed_args.name
@@ -164,32 +178,31 @@ def take_action(self, parsed_args):
         if 'enabled' in parsed_args:
             kwargs['enabled'] = parsed_args.enabled
 
-        if kwargs == {}:
-            sys.stdout.write("Tenant not updated, no arguments present")
-            return 0
-        tenant.update(**kwargs)
+        project.update(**kwargs)
         return
 
 
-class ShowTenant(show.ShowOne):
-    """Show tenant command"""
+class ShowProject(show.ShowOne):
+    """Show project details"""
 
-    log = logging.getLogger(__name__ + '.ShowTenant')
+    log = logging.getLogger(__name__ + '.ShowProject')
 
     def get_parser(self, prog_name):
-        parser = super(ShowTenant, self).get_parser(prog_name)
+        parser = super(ShowProject, self).get_parser(prog_name)
         parser.add_argument(
-            'tenant',
-            metavar='<tenant>',
-            help='Name or ID of tenant to display')
+            'project',
+            metavar='<project>',
+            help='Project to display (name or ID)')
         return parser
 
     def take_action(self, parsed_args):
         self.log.debug('take_action(%s)' % parsed_args)
         identity_client = self.app.client_manager.identity
-        tenant = utils.find_resource(identity_client.tenants,
-                                     parsed_args.tenant)
+        project = utils.find_resource(
+            identity_client.tenants,
+            parsed_args.project,
+        )
 
         info = {}
-        info.update(tenant._info)
+        info.update(project._info)
         return zip(*sorted(six.iteritems(info)))

openstackclient/identity/v2_0/role.py

@@ -13,7 +13,7 @@
 #   under the License.
 #
 
-"""Role action implementations"""
+"""Identity v2 Role action implementations"""
 
 import logging
 import six
@@ -26,7 +26,7 @@
 
 
 class AddRole(show.ShowOne):
-    """Add role to tenant:user"""
+    """Add role to project:user"""
 
     log = logging.getLogger(__name__ + '.AddRole')
 
@@ -37,10 +37,11 @@ def get_parser(self, prog_name):
             metavar='<role>',
             help='Role name or ID to add to user')
         parser.add_argument(
-            '--tenant',
-            metavar='<tenant>',
+            '--project',
+            metavar='<project>',
             required=True,
-            help='Name or ID of tenant to include')
+            help='Include project (name or ID)',
+        )
         parser.add_argument(
             '--user',
             metavar='<user>',
@@ -52,13 +53,15 @@ def take_action(self, parsed_args):
         self.log.debug('take_action(%s)' % parsed_args)
         identity_client = self.app.client_manager.identity
         role = utils.find_resource(identity_client.roles, parsed_args.role)
-        tenant = utils.find_resource(identity_client.tenants,
-                                     parsed_args.tenant)
+        project = utils.find_resource(
+            identity_client.tenants,
+            parsed_args.project,
+        )
         user = utils.find_resource(identity_client.users, parsed_args.user)
         role = identity_client.roles.add_user_role(
             user,
             role,
-            tenant)
+            project)
 
         info = {}
         info.update(role._info)
@@ -138,34 +141,43 @@ def get_parser(self, prog_name):
             nargs='?',
             help='Name or ID of user to include')
         parser.add_argument(
-            '--tenant',
-            metavar='<tenant>',
-            help='Name or ID of tenant to include')
+            '--project',
+            metavar='<project>',
+            help='Include project (name or ID)',
+        )
         return parser
 
     def take_action(self, parsed_args):
         self.log.debug('take_action(%s)' % parsed_args)
-        columns = ('ID', 'Name', 'Tenant ID', 'User ID')
         identity_client = self.app.client_manager.identity
 
         # user-only roles are not supported in KSL so we are
-        # required to have a user and tenant; default to the
+        # required to have a user and project; default to the
         # values used for authentication if not specified
-        if not parsed_args.tenant:
-            parsed_args.tenant = identity_client.auth_tenant_id
+        if not parsed_args.project:
+            parsed_args.project = identity_client.auth_tenant_id
         if not parsed_args.user:
             parsed_args.user = identity_client.auth_user_id
 
-        tenant = utils.find_resource(identity_client.tenants,
-                                     parsed_args.tenant)
+        project = utils.find_resource(
+            identity_client.tenants,
+            parsed_args.project,
+        )
         user = utils.find_resource(identity_client.users, parsed_args.user)
 
-        data = identity_client.roles.roles_for_user(user.id, tenant.id)
+        data = identity_client.roles.roles_for_user(user.id, project.id)
+
+        columns = (
+            'ID',
+            'Name',
+            'Project',
+            'User',
+        )
 
         # Add the names to the output even though they will be constant
         for role in data:
-            role.user_id = user.name
-            role.tenant_id = tenant.name
+            role.user = user.name
+            role.project = project.name
 
         return (columns,
                 (utils.get_item_properties(
@@ -175,7 +187,7 @@ def take_action(self, parsed_args):
 
 
 class RemoveRole(command.Command):
-    """Remove role from tenant:user"""
+    """Remove role from project:user"""
 
     log = logging.getLogger(__name__ + '.RemoveRole')
 
@@ -186,10 +198,11 @@ def get_parser(self, prog_name):
             metavar='<role>',
             help='Role name or ID to remove from user')
         parser.add_argument(
-            '--tenant',
-            metavar='<tenant>',
+            '--project',
+            metavar='<project>',
             required=True,
-            help='Name or ID of tenant')
+            help='Project to include (name or ID)',
+        )
         parser.add_argument(
             '--user',
             metavar='<user>',
@@ -201,13 +214,15 @@ def take_action(self, parsed_args):
         self.log.debug('take_action(%s)' % parsed_args)
         identity_client = self.app.client_manager.identity
         role = utils.find_resource(identity_client.roles, parsed_args.role)
-        tenant = utils.find_resource(identity_client.tenants,
-                                     parsed_args.tenant)
+        project = utils.find_resource(
+            identity_client.tenants,
+            parsed_args.project,
+        )
         user = utils.find_resource(identity_client.users, parsed_args.user)
         identity_client.roles.remove_user_role(
             user.id,
             role.id,
-            tenant.id)
+            project.id)
 
 
 class ShowRole(show.ShowOne):