Bug 1192950

Summary:	RHEVH Registration to default cluster fails when cluster Default does not exist or UUID is wrong
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	Yaniv Bronhaim <ybronhei>
Component:	ovirt-engine	Assignee:	Yaniv Bronhaim <ybronhei>
Status:	CLOSED CURRENTRELEASE	QA Contact:	movciari
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	3.6.0	CC:	emesika, gklein, lpeer, lsurette, oourfali, pstehlik, rbalakri, Rhev-m-bugs, sherold, srevivo, ybronhei, ykaul
Target Milestone:	ovirt-3.6.0-rc
Target Release:	3.6.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-04-20 01:37:13 UTC	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:

Description Yaniv Bronhaim 2015-02-16 10:21:10 UTC

1. the value AutoRegistrationDefaultVdsGroupID needs to get valid cluster id by default.

2. counting on cluster "default" can lead to unclear error if user deletes this cluster - if cluster id does not exist addVdsCommand will receive groupid null and will fail the canDoAction - it's are to understand from the log why the failure occurred. I suggest to add more info to the audit log.
 
3. rhevh does not expose auto registration option in TUI afaik. and the config value AutoApprovePatterns not so clear (at least to me)

In this bug scope I'd like to understand how the auto approval should work and be exposed to the user. for instance - the default cluster for auto approval should be set somewhere in the ui as global value imo

Comment 1 Eli Mesika 2015-02-16 10:42:27 UTC

(In reply to Yaniv Bronhaim from comment #0)
> 1. the value AutoRegistrationDefaultVdsGroupID needs to get valid cluster id
> by default.

addressed by gerrit.ovirt.org/#/c/37813

Comment 2 Eli Mesika 2015-02-17 09:10:50 UTC

(In reply to Yaniv Bronhaim from comment #0)

> 2. counting on cluster "default" can lead to unclear error if user deletes
> this cluster - if cluster id does not exist addVdsCommand will receive
> groupid null and will fail the canDoAction - it's are to understand from the
> log why the failure occurred. I suggest to add more info to the audit log.
>  

addressed by gerrit.ovirt.org/#/c/37842

Comment 3 Eli Mesika 2015-02-17 09:12:49 UTC

Yaniv , can we move that to POST after 1) and 2) were resolved (I think 3) deserve a separate issue that is more to the long term and 1) and 2) can be applied to 3.6)

Comment 4 Yaniv Bronhaim 2015-02-17 09:56:49 UTC

1,2 relates also to regular registration in PendingApproval state, the auto approval verification can be done separately . I'll open another bug regarding that.