On Mon, Aug 17, 2015 at 10:52:41AM +0200, Lukas Slebodnik wrote:
> On (17/08/15 10:32), Jakub Hrozek wrote:
> >On Sun, Aug 16, 2015 at 05:59:22PM +0200, Pavel Březina wrote:
> >> https://fedorahosted.org/sssd/ticket/2737
> >> 
> >> Hi, it should work... :-) However, I wanted to make import as transaction 
> >> so
> >> no changes are made if some error occurs, but I had some troubles with it 
> >> so
> >> I gave up eventually.
> >
> >Thanks a lot for the patches. I didn't do any review yet, just scrolled
> >through them.
> >
> >My first reaction was that the second patch should be split up and at
> >least the colondb should be unit tested.
> >
> >btw did you get inspired by some existing code, like libuser or
> >shadow-utils with the colondb?
> >
> Is there a design page for this?
> Maybe I missed something but I haven't noticed any discussion about
> export format for "local views".
> 
> I'm not very happy with custom format. Yes, it's already late
> because patches are ready and deadline is comming.

btw you're right there was no design page. But basically, we should
support this:
    https://support.software.dell.com/authentication-services/kb/91145
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to