On (12/07/16 18:38), Lukas Slebodnik wrote:
On (12/07/16 15:59), Michal Židek wrote:
>On 07/12/2016 03:36 PM, Lukas Slebodnik wrote:
>> On (12/07/16 15:16), Michal Židek wrote:
>> > +# secrets responder
>> > +option = provider
>> > +
>>
>> I think you need to also update "rule/allowed_sections"
>>
>> maybe you could run tour tool "sssctl config-check"
>> before sending patches :-)
>>
>
>What a useful tool it turned out to be :)
>
>> And there are another related question to this topic.
>> Should we add undocumented option to the list?
>>
>> We already have "command" in schema. Should we add other as well?
>>
>> IMHO, no.
>>
>> LS
>
>So far we only added options that we expect users
>to use. Options that are for developers are not added to the
>schema for now.
>
>Michal
>From 42a3038b68452cf92b2f87ae0875f4e3b8b1f051 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Michal=20=C5=BDidek?= <mzidek(a)redhat.com>
>Date: Mon, 11 Jul 2016 13:03:28 +0200
>Subject: [PATCH 1/3] config: Allow timeout for all sevices
>
>Fixes:
>https://fedorahosted.org/sssd/ticket/3068
>
>Allow option "timeout" for all sevices.
>Also remove unused macro CONFDB_SERVICE_TIMEOUT.
>---
ACK
>From cacd9f84e702c2aa7f5c41d0d257eb5ce8c77a12 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Michal=20=C5=BDidek?= <mzidek(a)redhat.com>
>Date: Mon, 11 Jul 2016 13:34:03 +0200
>Subject: [PATCH 2/3] config: Add config_file_version to schema
>
>Fixes:
>https://fedorahosted.org/sssd/ticket/3068
>---
> src/config/SSSDConfigTest.py | 1 +
> src/config/cfg_rules.ini | 1 +
> src/config/etc/sssd.api.conf | 1 +
> 3 files changed, 3 insertions(+)
>
ACK
http://sssd-ci.duckdns.org/logs-test/job/2/93/summary.html
1st and 2nd patch pushed to
master:
* aeab20358006d728a284f969f92f3890498cd651
* 1b9b5477027d86a2afb2e72981253d108c5398da
LS