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?= <mzi...@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?= <mzi...@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 _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org