[SSSD] [PATCH] CONFDB: Assume config file version 2 if missing

Michal Židek mzidek at redhat.com
Mon Jul 27 13:58:55 UTC 2015


On 07/24/2015 09:51 PM, Lukas Slebodnik wrote:
> On (07/07/15 19:45), Michal Židek wrote:
>> Hi!
>>
>> See the attached patch.
>>
>> Ticket:
>> https://fedorahosted.org/sssd/ticket/2688
>>
>> CI passed:
>> http://sssd-ci.duckdns.org/logs/job/18/71/summary.html
>>
>> Thanks,
>> Michal
>>
>> --
>> Senior Principal Intern
>
>>From c67f8df99df4f1aafa41d57a84c8d1c19fb5d610 Mon Sep 17 00:00:00 2001
>> From: =?UTF-8?q?Michal=20=C5=BDidek?= <mzidek at redhat.com>
>> Date: Tue, 7 Jul 2015 15:15:32 +0200
>> Subject: [PATCH] CONFDB: Assume config file version 2 if missing
>>
>> Default to config file version 2 if the version
>> is not specified explicitly.
>>
>> Ticket:
>> https://fedorahosted.org/sssd/ticket/2688
>> ---
>> src/confdb/confdb.h                          |  1 +
>> src/confdb/confdb_setup.c                    | 48 ++++++++++++++--------------
>> src/config/SSSDConfig/sssd_upgrade_config.py |  3 +-
>> 3 files changed, 27 insertions(+), 25 deletions(-)
>>
> I think you can also remove config_file_version from
> sssd.conf in integration test. Or at least comment it out.
> So it will be proven that your patch works.
>
> LS

Ok, I added patch that removes the option from
sssd.conf in integration tests.

Michal

-- 
Senior Principal Intern
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-CONFDB-Assume-config-file-version-2-if-missing.patch
Type: text/x-patch
Size: 3942 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20150727/ca5b391b/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-tests-Remove-config_file_version-from-sssd.conf-in-C.patch
Type: text/x-patch
Size: 1245 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20150727/ca5b391b/attachment-0001.bin>


More information about the sssd-devel mailing list