Bug 765907 - Pool owner perm should also grant pool user.. no need to list it twice
Summary: Pool owner perm should also grant pool user.. no need to list it twice
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-conductor
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: rc
Assignee: Scott Seago
QA Contact: Aziza Karol
URL: https://dhcp231-79.rdu.redhat.com/con...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-09 16:43 UTC by wes hayutin
Modified: 2012-05-15 21:28 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-05-15 21:28:39 UTC


Attachments (Terms of Use)
ss (182.89 KB, image/png)
2011-12-09 16:43 UTC, wes hayutin
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2012:0583 0 normal SHIPPED_LIVE new packages: aeolus-conductor 2012-05-15 22:31:59 UTC

Description wes hayutin 2011-12-09 16:43:26 UTC
Created attachment 544625 [details]
ss

Description of problem:

I would think that pool owner also grants pool user...
the app lists the user twice and the user count increments for each.. which is also probably not good...

see screenshot

Comment 1 Scott Seago 2011-12-13 16:09:16 UTC
So there are 2 things going on here:

1) roles are independently-defined. A role is a collection of lower-level privileges combined together. In some case, like here, one role is a subset of another, but in other cases they are not (i.e. "Provider Admin" and "Pool Admin" on the global roles).

More specifically, Pool Owner doesn't grant Pool User -- rather Pool Owner and Pool User both grant the privilege of viewing pools and launching instances in them.

2) the tab was originally called "Permissions" this was changed to "Users" by the UX team. While it is incorrect that there are "2 users", there are, in fact, "2 permissions grants". Specifically, any time a user is validly added to more than one role the number won't match the number of users, as it's counting permission grants. This will be even more confusing once we start allowing permission grants to user groups here.

So, as for the correct resolution:
1) we had talked about preventing a user from being assigned more than one role _if_ one was a subset of the other. The downside is we'd have to pull the privilege list for each role and cross-reference all of them across all roles to determine what to hide. I'm not sure that's a great idea here.

2) Perhaps we should re-think calling this "Users" and go back to "Permission grants" or "Role assignments" (but _not_ just "Roles")

Comment 2 Angus Thomas 2011-12-13 18:15:31 UTC
FWIW, I think that preventing users from being assigned multiple roles, where one if effectively a subset of the other, given our intention to support an arbitrary number of user-defined roles in future, would be complex adn unintuitive for the user.

Changing the name to "Role Assignments" would be beneficial, as it more accurately describes the objects which the user is presented with.

Comment 3 Scott Seago 2011-12-13 20:23:40 UTC
Patch posted: https://fedorahosted.org/pipermail/aeolus-devel/2011-December/007400.html

Permissions tab label updated from 'Users' to 'Role Assignments'

No changes to multiple role assignment behavior, in agreement with Angus above.

Comment 4 Scott Seago 2011-12-13 23:34:11 UTC
Commit pushed to master.

Comment 5 Steve Linabery 2012-01-10 17:30:29 UTC
245fa3e6a6a97402c4aa3744a872d08e51cfa608 in aeolus-conductor-0.8.0-2.el6.src.rpm

Comment 6 wes hayutin 2012-01-12 16:15:10 UTC
bugs in verified or on_qa moving off tracker

Comment 7 Aziza Karol 2012-01-20 04:54:04 UTC
Permissions tab label is updated from 'Users' to 'Role Assignments'.

In agreement with Angus from comment 2,no changes to multiple role assignment behaviour.

verified on:
 rpm -qa | grep aeolus
rubygem-aeolus-image-0.3.0-3.el6.noarch
rubygem-aeolus-cli-0.3.0-4.el6.noarch
aeolus-conductor-doc-0.8.0-8.el6.noarch
aeolus-configure-2.5.0-5.el6.noarch
aeolus-conductor-daemons-0.8.0-8.el6.noarch
aeolus-all-0.8.0-8.el6.noarch
aeolus-conductor-0.8.0-8.el6.noarch

Comment 9 errata-xmlrpc 2012-05-15 21:28:39 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2012-0583.html


Note You need to log in before you can comment on or make changes to this bug.