Bug 908246

Summary:	messages regarding permissions at manage-domains should be more clear
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	Yair Zaslavsky <yzaslavs>
Component:	ovirt-engine-config	Assignee:	Ravi Nori <rnori>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Ilanit Stein <istein>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	3.2.0	CC:	acathrow, bazulay, dyasny, iheim, oourfali, oramraz, Rhev-m-bugs, sgrinber, ykaul
Target Milestone:	---
Target Release:	3.2.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	infra
Fixed In Version:	sf8	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Infra	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	915537

Description Yair Zaslavsky 2013-02-06 08:59:12 UTC

Description of problem:

Two use cases:
1. User adds a domain with no "-addPermissions" -

The message he gets is:

"The domain example.com has been added to the engine as an authentication source but no users from that domain have been granted permissions within the oVirt Manager.
Users from this domain can be granted permissions from the Web administration interface."

I suggest to elaborate here and add it is possible to edit the domain and addPermissions (with action=edit and -addPermissions")


2. User adds a domain with "-addPermissions"
After that , let's say the user needs to change password, he uses action=edit, but without addPermissiosn (as they were already added) .

The user gets the same message from above, but this is incorrect, as there is a user from the domain with associated permissions.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 2 Ilanit Stein 2013-03-21 11:15:10 UTC

Verified on sf-10:

case 1)
[root@istein-32 ~]# rhevm-manage-domains  -action=add -domain=qa.lab.tlv.redhat.com -user=vdcadmin  -interactive -provider=activeDirectory
Enter password:

The domain qa.lab.tlv.redhat.com has been added to the engine as an authentication source but no users from that domain have been granted permissions within the oVirt Manager.
Users from this domain can be granted permissions from the Web administration interface or by editing the domain using -action=edit and specifying -addPermissions.
oVirt Engine restart is required in order for the changes to take place (service ovirt-engine restart).
Manage Domains completed successfully

[root@istein-32 ~]# service ovirt-engine restart
Stopping engine-service:                                    [  OK  ]
Starting engine-service:                                    [  OK  ]

[root@istein-32 ~]# rhevm-manage-domains  -action=edit -domain=qa.lab.tlv.redhat.com -user=vdcadmin  -interactive -provider=activeDirectory  -addPermissions 
Enter password:

Successfully added domain qa.lab.tlv.redhat.com. oVirt Engine restart is required in order for the changes to take place (service ovirt-engine restart).
Manage Domains completed successfully

case 2)
[root@istein-32 ~]# rhevm-manage-domains  -action=add -domain=qa.lab.tlv.redhat.com -user=vdcadmin  -interactive -provider=activeDirectory -addPermissions
Enter password:
Successfully added domain qa.lab.tlv.redhat.com. oVirt Engine restart is required in order for the changes to take place (service ovirt-engine restart).
Manage Domains completed successfully
[root@istein-32 ~]# service ovirt-engine restart
Stopping engine-service:                                    [  OK  ]
Starting engine-service:                                    [  OK  ]
[root@istein-32 ~]# rhevm-manage-domains  -action=edit -domain=qa.lab.tlv.redhat.com -user=vdcadmin  -interactive -provider=activeDirectory  

Enter password:

oVirt Engine restart is required in order for the changes to take place (service ovirt-engine restart).
Manage Domains completed successfully

Comment 3 Itamar Heim 2013-06-11 09:28:12 UTC

3.2 has been released

Comment 4 Itamar Heim 2013-06-11 09:31:39 UTC

3.2 has been released

Comment 5 Itamar Heim 2013-06-11 09:47:28 UTC

3.2 has been released