Bug 885766

Summary:	engine-manage-domains -addPermission doesn't work on f18
Product:	[Retired] oVirt	Reporter:	Ohad Basan <obasan>
Component:	ovirt-engine-config	Assignee:	Ravi Nori <rnori>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	unspecified	Docs Contact:
Priority:	urgent
Version:	3.1 RC	CC:	acathrow, dyasny, eedri, iheim, mgoldboi, ykaul, yzaslavs
Target Milestone:	---
Target Release:	3.2
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	infra
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-02-15 06:47:56 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Ohad Basan 2012-12-10 15:16:33 UTC

Description of problem:
the -addpermissions doesn't work on fedora 18. the user's entry isn't updated in the permissions table

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


How reproducible:
always

Steps to Reproduce:
1.install ovirt-engine nightly on fedora18
2.run engine-manage-domains with the -user and -addpermissions parameter
3.log in with admin@internal 
4. go to "configure"> system permissions

  
Actual results:
the user we've added in the engine-manage-domain commands isn't being given with the permissions

Expected results:
user should be granted with permissions


Additional info:

Comment 1 Yair Zaslavsky 2012-12-11 08:21:53 UTC

What did the utility print you? it print you success?
I managed to add an IPA domain and saw the permissions table getting filled.
Did you try with IPA or ActiveDirectory?
Did you check the permissions table is filled?

Comment 2 Yair Zaslavsky 2012-12-11 17:26:10 UTC

http://gerrit.ovirt.org/gitweb?p=ovirt-engine.git;a=commit;h=545a14725b033a24be4dadd4fde8ee2de1c71053

Comment 3 Yair Zaslavsky 2012-12-11 17:27:02 UTC

(Turns out this was an issue with passing permissionId as parameter to stored procedure on manage domains. SqlType.Other must be used).

Comment 4 Itamar Heim 2013-01-16 16:10:31 UTC

3.2 beta built, moving to ON_QA status to allow testing