| 2012-11-02 02:08:56 |
ryuta sekine |
bug |
|
|
added bug |
| 2012-11-02 02:59:45 |
ryuta sekine |
description |
Tenant ID is a required parameter of user-role-add, but command display is optional parameter.
The following is the display of when the keystone user-role-add command is executed.
usage: keystone user-role-add --user-id <user-id> --role-id <role-id>
[--tenant-id <tenant-id>]
keystone user-role-add: error: argument --user-id/--user_id is required
Tenant ID should fix to a required parameter display, because it is an error if you do not specify tenant ID. |
Tenant ID is a required parameter of user-role-add, but command display is optional parameter.
The following is the display of when the keystone user-role-add command is executed.
usage: keystone user-role-add --user-id <user-id> --role-id <role-id>
[--tenant-id <tenant-id>]
keystone user-role-add: error: argument --user-id/--user_id is required
In /opt/stack/keystone/keystone/identity/core.py
def add_role_to_user(self, context, user_id, role_id, tenant_id=None):
"""Add a role to a user and tenant pair.
Since we're trying to ignore the idea of user-only roles we're
not implementing them in hopes that the idea will die off.
"""
self.assert_admin(context)
if tenant_id is None:
raise exception.NotImplemented(message='User roles not supported: '
'tenant_id required')
tenant ID should fix to a required parameter display, because it is an error if you do not specify tenant ID. |
|
| 2012-11-02 04:11:28 |
ryuta sekine |
description |
Tenant ID is a required parameter of user-role-add, but command display is optional parameter.
The following is the display of when the keystone user-role-add command is executed.
usage: keystone user-role-add --user-id <user-id> --role-id <role-id>
[--tenant-id <tenant-id>]
keystone user-role-add: error: argument --user-id/--user_id is required
In /opt/stack/keystone/keystone/identity/core.py
def add_role_to_user(self, context, user_id, role_id, tenant_id=None):
"""Add a role to a user and tenant pair.
Since we're trying to ignore the idea of user-only roles we're
not implementing them in hopes that the idea will die off.
"""
self.assert_admin(context)
if tenant_id is None:
raise exception.NotImplemented(message='User roles not supported: '
'tenant_id required')
tenant ID should fix to a required parameter display, because it is an error if you do not specify tenant ID. |
Tenant ID is a required parameter of user-role-add, but command display is optional parameter.
The following is the display of when the keystone user-role-add command is executed.
usage: keystone user-role-add --user-id <user-id> --role-id <role-id>
[--tenant-id <tenant-id>]
keystone user-role-add: error: argument --user-id/--user_id is required
In /opt/stack/keystone/keystone/identity/core.py
def add_role_to_user(self, context, user_id, role_id, tenant_id=None):
"""Add a role to a user and tenant pair.
Since we're trying to ignore the idea of user-only roles we're
not implementing them in hopes that the idea will die off.
"""
self.assert_admin(context)
if tenant_id is None:
raise exception.NotImplemented(message='User roles not supported: '
'tenant_id required')
Because not yet implemented it return an error if you do not specify tenant ID.
This case tenant ID should fix to a required parameter display. |
|
| 2012-11-06 04:56:41 |
Joseph Heck |
python-keystoneclient: status |
New |
Triaged |
|
| 2012-11-06 04:56:44 |
Joseph Heck |
python-keystoneclient: importance |
Undecided |
High |
|
| 2012-11-13 09:11:21 |
OpenStack Infra |
python-keystoneclient: status |
Triaged |
In Progress |
|
| 2012-11-13 09:11:21 |
OpenStack Infra |
python-keystoneclient: assignee |
|
Andre Naehring (naehring) |
|
| 2012-11-13 09:18:23 |
Andre Naehring |
python-keystoneclient: status |
In Progress |
Fix Committed |
|
| 2012-11-13 09:18:29 |
Andre Naehring |
python-keystoneclient: status |
Fix Committed |
In Progress |
|
| 2012-11-13 13:30:07 |
Dolph Mathews |
python-keystoneclient: status |
In Progress |
Invalid |
|
