|
 |
bb7cd1 |
From b818bb3f27ce672df0a6cadf2fd90716d2a576dc Mon Sep 17 00:00:00 2001
|
|
|
bb7cd1 |
From: Justin Stephenson <jstephen@redhat.com>
|
|
|
bb7cd1 |
Date: Wed, 26 Apr 2017 15:45:33 -0400
|
|
|
bb7cd1 |
Subject: [PATCH 122/127] DP: Reduce Data Provider log level noise
|
|
|
bb7cd1 |
MIME-Version: 1.0
|
|
|
bb7cd1 |
Content-Type: text/plain; charset=UTF-8
|
|
|
bb7cd1 |
Content-Transfer-Encoding: 8bit
|
|
|
bb7cd1 |
|
|
|
bb7cd1 |
Certain operations are not supported with certain providers
|
|
|
bb7cd1 |
causing informational Data Provider log messages to be logged as
|
|
|
bb7cd1 |
errors or failures. This patch lowers the log level to reduce overall
|
|
|
bb7cd1 |
log noise and ensure only critical log messages are logged when
|
|
|
bb7cd1 |
a low debug_level value is used.
|
|
|
bb7cd1 |
|
|
|
bb7cd1 |
Resolves:
|
|
|
bb7cd1 |
https://pagure.io/SSSD/sssd/issue/3287
|
|
|
bb7cd1 |
https://pagure.io/SSSD/sssd/issue/3278
|
|
|
bb7cd1 |
|
|
|
bb7cd1 |
Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>
|
|
|
bb7cd1 |
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
|
|
|
bb7cd1 |
(cherry picked from commit e98d085b529e0ae5e07a717ce3b30f3943be0ee0)
|
|
|
bb7cd1 |
---
|
|
|
bb7cd1 |
src/providers/data_provider/dp_methods.c | 2 +-
|
|
|
bb7cd1 |
src/providers/data_provider/dp_targets.c | 2 +-
|
|
|
bb7cd1 |
src/responder/common/responder_dp.c | 13 +++++++++++--
|
|
|
bb7cd1 |
3 files changed, 13 insertions(+), 4 deletions(-)
|
|
|
bb7cd1 |
|
|
|
bb7cd1 |
diff --git a/src/providers/data_provider/dp_methods.c b/src/providers/data_provider/dp_methods.c
|
|
|
bb7cd1 |
index 498676d1bec2da300ca4b33f7110debcbf0aac00..9e49c5f5d65b869b3699fdc682a535e0111b6fd4 100644
|
|
|
bb7cd1 |
--- a/src/providers/data_provider/dp_methods.c
|
|
|
bb7cd1 |
+++ b/src/providers/data_provider/dp_methods.c
|
|
|
bb7cd1 |
@@ -109,7 +109,7 @@ errno_t dp_find_method(struct data_provider *provider,
|
|
|
bb7cd1 |
}
|
|
|
bb7cd1 |
|
|
|
bb7cd1 |
if (!dp_target_initialized(provider->targets, target)) {
|
|
|
bb7cd1 |
- DEBUG(SSSDBG_CRIT_FAILURE, "Target [%s] is not initialized\n",
|
|
|
bb7cd1 |
+ DEBUG(SSSDBG_CONF_SETTINGS, "Target [%s] is not initialized\n",
|
|
|
bb7cd1 |
dp_target_to_string(target));
|
|
|
bb7cd1 |
return ERR_MISSING_DP_TARGET;
|
|
|
bb7cd1 |
}
|
|
|
bb7cd1 |
diff --git a/src/providers/data_provider/dp_targets.c b/src/providers/data_provider/dp_targets.c
|
|
|
bb7cd1 |
index 26d20a8ef79b80d56df76d7a73ec8e63d001ecbc..e2a45bbac969ca7b9b13729f26b8cded8ab7eebc 100644
|
|
|
bb7cd1 |
--- a/src/providers/data_provider/dp_targets.c
|
|
|
bb7cd1 |
+++ b/src/providers/data_provider/dp_targets.c
|
|
|
bb7cd1 |
@@ -284,7 +284,7 @@ static errno_t dp_target_init(struct be_ctx *be_ctx,
|
|
|
bb7cd1 |
if (!target->explicitly_configured && (ret == ELIBBAD || ret == ENOTSUP)) {
|
|
|
bb7cd1 |
/* Target not found but it wasn't explicitly
|
|
|
bb7cd1 |
* configured so we shall just continue. */
|
|
|
bb7cd1 |
- DEBUG(SSSDBG_CRIT_FAILURE, "Target [%s] is not supported by "
|
|
|
bb7cd1 |
+ DEBUG(SSSDBG_CONF_SETTINGS, "Target [%s] is not supported by "
|
|
|
bb7cd1 |
"module [%s].\n", target->name, target->module_name);
|
|
|
bb7cd1 |
ret = EOK;
|
|
|
bb7cd1 |
goto done;
|
|
|
bb7cd1 |
diff --git a/src/responder/common/responder_dp.c b/src/responder/common/responder_dp.c
|
|
|
bb7cd1 |
index 080f70fd5945ffd234e0ef226d8139df071c4752..a75a611960801f5f5bdc95f00aea9ab921e8e293 100644
|
|
|
bb7cd1 |
--- a/src/responder/common/responder_dp.c
|
|
|
bb7cd1 |
+++ b/src/responder/common/responder_dp.c
|
|
|
bb7cd1 |
@@ -218,8 +218,17 @@ static int sss_dp_get_reply(DBusPendingCall *pending,
|
|
|
bb7cd1 |
err = ETIME;
|
|
|
bb7cd1 |
goto done;
|
|
|
bb7cd1 |
}
|
|
|
bb7cd1 |
- DEBUG(SSSDBG_FATAL_FAILURE,"The Data Provider returned an error [%s]\n",
|
|
|
bb7cd1 |
- dbus_message_get_error_name(reply));
|
|
|
bb7cd1 |
+
|
|
|
bb7cd1 |
+ if (strcmp(dbus_message_get_error_name(reply),
|
|
|
bb7cd1 |
+ SBUS_ERROR_DP_NOTSUP) == 0) {
|
|
|
bb7cd1 |
+ DEBUG(SSSDBG_CONF_SETTINGS,
|
|
|
bb7cd1 |
+ "Data Provider does not support this operation.\n");
|
|
|
bb7cd1 |
+ } else {
|
|
|
bb7cd1 |
+ DEBUG(SSSDBG_FATAL_FAILURE,
|
|
|
bb7cd1 |
+ "The Data Provider returned an error [%s]\n",
|
|
|
bb7cd1 |
+ dbus_message_get_error_name(reply));
|
|
|
bb7cd1 |
+ }
|
|
|
bb7cd1 |
+
|
|
|
bb7cd1 |
/* Falling through to default intentionally*/
|
|
|
bb7cd1 |
SSS_ATTRIBUTE_FALLTHROUGH;
|
|
|
bb7cd1 |
default:
|
|
|
bb7cd1 |
--
|
|
|
bb7cd1 |
2.9.3
|
|
|
bb7cd1 |
|