[SSSD] [PATCH] Update few debug messages

Lukas Slebodnik lslebodn at redhat.com
Thu Jun 18 16:16:36 UTC 2015


ehlo,

I noticed the same noise in log files as user in ticker 2678.
It was a little bit related to the latest changes with setting
initgroups flag in the right time.

LS
-------------- next part --------------
>From b0ab92f873a9ee694c31c735a60ee1e0e66a7dc2 Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik <lslebodn at redhat.com>
Date: Thu, 18 Jun 2015 17:10:21 +0200
Subject: [PATCH 2/2] Update few debug messages

It reduces a noise caused by canonicalization of non-existing user.

Resolves:
https://fedorahosted.org/sssd/ticket/2678
---
 src/db/sysdb_search.c        | 7 ++++---
 src/providers/ldap/ldap_id.c | 5 +++--
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/db/sysdb_search.c b/src/db/sysdb_search.c
index a8dcc9f8d6617be8e8fb82a1c6360c6df9726a37..c92474875a07059abff27e8d329a24187dde97a2 100644
--- a/src/db/sysdb_search.c
+++ b/src/db/sysdb_search.c
@@ -1613,10 +1613,11 @@ errno_t sysdb_get_real_name(TALLOC_CTX *mem_ctx,
                                                   &res);
                 if (ret == EOK && res->count == 1) {
                     msg = res->msgs[0];
-                } else {
+                } else if (ret != ENOENT) {
                     DEBUG(SSSDBG_OP_FAILURE,
-                          "sysdb_search_object_by_uuid did not return a " \
-                          "single result.\n");
+                          "sysdb_search_object_by_uuid failed or returned "
+                          "more than one result [%d][%s].\n",
+                          ret, strerror(ret));
                     ret = ENOENT;
                     goto done;
                 }
diff --git a/src/providers/ldap/ldap_id.c b/src/providers/ldap/ldap_id.c
index 1676b8d5c1b4dc8875dc642fb5f8776c1879a655..ac01bec0a8f1196c93971a7c11432e998c224478 100644
--- a/src/providers/ldap/ldap_id.c
+++ b/src/providers/ldap/ldap_id.c
@@ -1154,8 +1154,9 @@ static void groups_by_user_done(struct tevent_req *subreq)
     ret = sysdb_get_real_name(state, state->domain, state->name, &cname);
     if (ret != EOK) {
         cname = state->name;
-        DEBUG(SSSDBG_OP_FAILURE, "Failed to canonicalize name, using [%s].\n",
-                                 cname);
+        DEBUG(SSSDBG_TRACE_INTERNAL,
+              "Failed to canonicalize name, using [%s] [%d]: %s.\n",
+              cname, ret, sss_strerror(ret));
     }
 
     switch (state->sdap_ret) {
-- 
2.4.3



More information about the sssd-devel mailing list