On 04/09/2012 12:11 PM, Livnat Peer wrote:
On 09/04/12 11:42, Roy Golan wrote:
> auto-detection featured few bugs lately which convinced me that it must
> be deprecated/removed from
> the code in favor of adding manual set of the provider type.
>
> main reasons for that:
> a. the provider is type is known by the user
> b. identifying a provider by the rootDSE attributes is error-prone, not
> reliable etc... specially if the support matrix is growing
> c. auto-detection logic is duplicated (and not shared) between the
> backend code and the manage-domains tool (nevermind the reasons now)
>
+1 for removing this 'feature'
> A manual set of the provider type requires a new configuration field for
> a domain and the
> question is the upgrade path which may look like
>
> * set the provider type to default Active Directory (because its more
> likely to be the provider type)
> * things not working - use the engine-manage-domains and choose the
> provider type.
> * optionally - keep the auto-detection (of the tool only!) if user
> wants to use it.
I think we should require that the user specifies the provider type
(from now on), and enable editing this field of course.
Another upgrade proposal is to populate the provider type only upon the
first time we need to use this domain (and it is not populated) by using
the current 'auto-detect' mechanism.
and remove it the next version?
this means the auto-detection code remains in the backend code.
I think its the best to remove it totally but If we want to keep this
functionality I rather just leave it in manage-domains alone,although if
its another step for the user.
By taking the above approach we don't break behavior for the user and we
enable manually fixing of the domain provider type (which is a bypass
for all current bugs in the auto-detect).
Livnat
> your suggestion and thoughts...
>
> Thanks,
> Roy
>
>
>
> _______________________________________________
> Engine-devel mailing list
> Engine-devel(a)ovirt.org
>
http://lists.ovirt.org/mailman/listinfo/engine-devel