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

From: "Livnat Peer" <lpeer(a)redhat.com&gt; To: "Roy Golan" <rgolan(a)redhat.com&gt; Cc: "Oved Ourfalli" <ovedo(a)redhat.com&gt;, engine-devel(a)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'

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