On 02/18/2015 09:45 AM, Lukas Slebodnik wrote:
On (17/02/15 16:43), Lukas Slebodnik wrote:
ehlo,

Attached patch should simplify analysis of log files.
It should be applied to 1-12 branch. I will generate another version for master
after merging all pending big patchsets.

LS
Attached is updated version which remove extra new line added
in logging of nsupdate message.

LS


_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
Patch fails to apply at master as you expected, so no CI has been run so far. But Nick is already working on CI support for 1-12.

Patch applies and builds fine on 1-12, I just noted one nitpick. But I think it is OK to ACK the patch as is.
+++ b/src/providers/ipa/ipa_hbac_rules.c
@@ -144,7 +144,7 @@ ipa_hbac_rule_info_send(TALLOC_CTX *mem_ctx,
     ret = sysdb_attrs_get_string_array(ipa_host, SYSDB_ORIG_MEMBEROF,
                                        tmp_ctx, &memberof_list);
     if (ret != EOK && ret != ENOENT) {
-        DEBUG(SSSDBG_CRIT_FAILURE, "Could not identify ");
+        DEBUG(SSSDBG_CRIT_FAILURE, "Could not identify \n");
space before '\n'
     } if (ret == ENOENT) {
         /* This host is not a member of any hostgroups */
         memberof_list = talloc_array(tmp_ctx, const char *, 1);

>From 7a25579801a72e93df4d1f9f18cee1db3388f8e0 Mon Sep 17 00:00:00 2001
From: Pavel Reichl <prei...@redhat.com>
Date: Wed, 18 Feb 2015 04:22:50 -0500
Subject: [PATCH] tmp

---
 src/providers/dp_dyndns.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/providers/dp_dyndns.c b/src/providers/dp_dyndns.c
index e5f3560c8397d0b9e5f6998796b3fc65a97ba1cc..a051a23a999e0b578181f980d9aea18edc96fa01 100644
--- a/src/providers/dp_dyndns.c
+++ b/src/providers/dp_dyndns.c
@@ -522,7 +522,6 @@ be_nsupdate_create_ptr_msg(TALLOC_CTX *mem_ctx, const char *realm,
           "%s"
           " -- End nsupdate message -- \n",
           update_msg);
-    DEBUG(SSSDBG_TRACE_FUNC,
 
     ret = ERR_OK;
     *_update_msg = talloc_steal(mem_ctx, update_msg);
-- 
2.1.0

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to