[SSSD] [PATCH] sss_override: support import and export

Lukas Slebodnik lslebodn at redhat.com
Mon Aug 17 08:52:41 UTC 2015


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.

Did you consider a ldif format?

libldb already provide such functions.
[alcik at unused-4-233][~/projects/sssd]$objdump -T /usr/lib64/libldb.so | grep ldb_ldif
000000000000afe0 g    DF .text  0000000000000641  LDB_1.1.0   ldb_ldif_parse_modrdn
000000000000afc0 g    DF .text  0000000000000008  LDB_0.9.10  ldb_ldif_write
000000000000bcc0 g    DF .text  000000000000000f  LDB_1.1.5   ldb_ldif_read_file_state
000000000000be00 g    DF .text  0000000000000019  LDB_0.9.10  ldb_ldif_message_string
000000000000bd40 g    DF .text  0000000000000054  LDB_1.1.12  ldb_ldif_write_redacted_trace_string
000000000000bcd0 g    DF .text  0000000000000015  LDB_0.9.10  ldb_ldif_read_file
000000000000bd20 g    DF .text  000000000000001f  LDB_0.9.10  ldb_ldif_write_file
000000000000b630 g    DF .text  0000000000000683  LDB_0.9.10  ldb_ldif_read
000000000000afd0 g    DF .text  000000000000000f  LDB_0.9.10  ldb_ldif_read_free
000000000000bda0 g    DF .text  0000000000000054  LDB_0.9.10  ldb_ldif_write_string
000000000000bcf0 g    DF .text  000000000000002b  LDB_0.9.10  ldb_ldif_read_string

e.g.
ldb_ldif_message_string can be used to format "struct ldb_message" as string
/*
   Produce a string form of an ldb message

   convenient function to turn a ldb_message into a string. Useful for
   debugging
 */
char *ldb_ldif_message_string(struct ldb_context *ldb, TALLOC_CTX *mem_ctx,
                              enum ldb_changetype changetype,
                              const struct ldb_message *msg);

LS



More information about the sssd-devel mailing list