[Engine-devel] deprecating auto-detection of Ldap provider types

[Oved Ourfalli]

----- Original Message -----
> From: "Livnat Peer" <lpeer at redhat.com>
> To: "Roy Golan" <rgolan at redhat.com>
> Cc: "Oved Ourfalli" <ovedo at redhat.com>, engine-devel at ovirt.org
> Sent: Monday, April 9, 2012 12:11:55 PM
> Subject: Re: [Engine-devel] deprecating auto-detection of Ldap provider types
> 
> 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'
> 
+1 for that as well.

> > 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.
> 
> 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 at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/engine-devel
> 
> _______________________________________________
> Engine-devel mailing list
> Engine-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/engine-devel
>