On (30/08/16 13:41), Lukas Slebodnik wrote:
On (30/08/16 13:13), Pavel Březina wrote:
>On 08/05/2016 05:24 PM, Lukas Slebodnik wrote:
>> On (04/08/16 14:11), Pavel Březina wrote:
>> > Simple one liner attached.
>>
>> > From 6c3d8d6dba969960590d88bdd9a4ec2e9b4a1bb2 Mon Sep 17 00:00:00 2001
>> > From: =?UTF-8?q?Pavel=20B=C5=99ezina?= <pbrezina(a)redhat.com>
>> > Date: Thu, 4 Aug 2016 14:10:09 +0200
>> > Subject: [PATCH] Fix typo in debug message
>> >
>> > ---
>> > src/providers/ipa/ipa_init.c | 2 +-
>> > 1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git a/src/providers/ipa/ipa_init.c b/src/providers/ipa/ipa_init.c
>> > index
ca99200a1ba1d9508ac0affffecaa08149552fee..7dec4d1fb8541a48470d4e44f10838e5bea67ad5 100644
>> > --- a/src/providers/ipa/ipa_init.c
>> > +++ b/src/providers/ipa/ipa_init.c
>> > @@ -200,7 +200,7 @@ static errno_t ipa_init_dyndns(struct be_ctx *be_ctx,
>> > enabled = dp_opt_get_bool(ipa_options->dyndns_ctx->opts,
>> > DP_OPT_DYNDNS_UPDATE);
>> > if (!enabled) {
>> > - DEBUG(SSSDBG_CONF_SETTINGS, "Dynamic DNS updates are
of.\n");
>> > + DEBUG(SSSDBG_CONF_SETTINGS, "Dynamic DNS updates are
off.\n");
>>
>> It would be good to have the same message as in ad provider.
>> Or change it also there :-)
>>
>> sh$ git grep "Dynamic DNS updates"
>> src/config/cfg_rules.ini:# Dynamic DNS updates
>> src/config/etc/sssd.api.conf:# Dynamic DNS updates
>> src/providers/ad/ad_common.h: /* Dynamic DNS updates */
>> src/providers/ad/ad_dyndns.c: DEBUG(SSSDBG_CONF_SETTINGS, "Dynamic
DNS updates not set\n");
>> src/providers/ad/ad_dyndns.c: "Dynamic DNS updates are on. Checking
for nsupdate..\n");
>> src/providers/be_dyndns.c: "%s does not exist. Dynamic DNS
updates disabled\n",
>> src/providers/ipa/ipa_init.c: DEBUG(SSSDBG_CONF_SETTINGS, "Dynamic
DNS updates are of.\n");
>> src/providers/ipa/ipa_init.c: "Dynamic DNS updates are on. Checking
for nsupdate...\n");
>>
>> LS
>>
>
>From 40850397be1e8be3b38437a6a93f4c1772c2f6af Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Pavel=20B=C5=99ezina?= <pbrezina(a)redhat.com>
>Date: Thu, 4 Aug 2016 14:10:09 +0200
>Subject: [PATCH] dyndns: fix typo and unify ipa with ad debug message when off
>
>---
> src/providers/ad/ad_dyndns.c | 2 +-
> src/providers/ipa/ipa_init.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/src/providers/ad/ad_dyndns.c b/src/providers/ad/ad_dyndns.c
>index
e3f1812837f7cee9d18ef001233871e0fcc16b4c..00190485e8f0ca7362ed60b2df022c74c53988c9 100644
>--- a/src/providers/ad/ad_dyndns.c
>+++ b/src/providers/ad/ad_dyndns.c
>@@ -47,7 +47,7 @@ errno_t ad_dyndns_init(struct be_ctx *be_ctx,
>
> if (dp_opt_get_bool(ad_opts->dyndns_ctx->opts,
> DP_OPT_DYNDNS_UPDATE) == false) {
>- DEBUG(SSSDBG_CONF_SETTINGS, "Dynamic DNS updates not set\n");
>+ DEBUG(SSSDBG_CONF_SETTINGS, "Dynamic DNS updates are off.\n");
> return EOK;
> }
>
>diff --git a/src/providers/ipa/ipa_init.c b/src/providers/ipa/ipa_init.c
>index
ca99200a1ba1d9508ac0affffecaa08149552fee..7dec4d1fb8541a48470d4e44f10838e5bea67ad5 100644
>--- a/src/providers/ipa/ipa_init.c
>+++ b/src/providers/ipa/ipa_init.c
>@@ -200,7 +200,7 @@ static errno_t ipa_init_dyndns(struct be_ctx *be_ctx,
> enabled = dp_opt_get_bool(ipa_options->dyndns_ctx->opts,
> DP_OPT_DYNDNS_UPDATE);
> if (!enabled) {
>- DEBUG(SSSDBG_CONF_SETTINGS, "Dynamic DNS updates are of.\n");
>+ DEBUG(SSSDBG_CONF_SETTINGS, "Dynamic DNS updates are off.\n");
> return EOK;
ACK
master:
* b3851e86af91dc1aa6e265d5b2e4279b2611ff43
LS