summaryrefslogtreecommitdiffstats
path: root/src/providers/data_provider_fo.c
diff options
context:
space:
mode:
authorLukas Slebodnik <lslebodn@redhat.com>2016-01-28 11:54:04 +0100
committerLukas Slebodnik <lslebodn@redhat.com>2016-01-28 14:42:01 +0100
commitb095ba05bf54def5b2dce6bb60946e272ad27cfc (patch)
treea061301e03b1382165ac4e6dd0e68920f783599d /src/providers/data_provider_fo.c
parent0f99ab0ad35700f7eff4809a7f4859635311ba6d (diff)
downloadsssd-b095ba05bf54def5b2dce6bb60946e272ad27cfc.tar.gz
sssd-b095ba05bf54def5b2dce6bb60946e272ad27cfc.tar.xz
sssd-b095ba05bf54def5b2dce6bb60946e272ad27cfc.zip
FAILOVER: Improve reporting of errors
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
Diffstat (limited to 'src/providers/data_provider_fo.c')
-rw-r--r--src/providers/data_provider_fo.c20
1 files changed, 13 insertions, 7 deletions
diff --git a/src/providers/data_provider_fo.c b/src/providers/data_provider_fo.c
index 39f57255a..38a23a23b 100644
--- a/src/providers/data_provider_fo.c
+++ b/src/providers/data_provider_fo.c
@@ -316,7 +316,8 @@ int be_fo_add_srv_server(struct be_ctx *ctx,
proto_table[proto], user_data);
if (ret && ret != EEXIST) {
DEBUG(SSSDBG_CRIT_FAILURE,
- "Failed to add SRV lookup reference to failover service\n");
+ "Failed to add SRV lookup reference to failover service "
+ "[%d]: %s\n", ret, sss_strerror(ret));
return ret;
}
@@ -329,7 +330,8 @@ int be_fo_add_srv_server(struct be_ctx *ctx,
proto_table[i], user_data);
if (ret && ret != EEXIST) {
DEBUG(SSSDBG_CRIT_FAILURE,
- "Failed to add SRV lookup reference to failover service\n");
+ "Failed to add SRV lookup reference to failover "
+ "service [%d]: %s\n", ret, sss_strerror(ret));
return ret;
}
@@ -368,7 +370,8 @@ int be_fo_add_server(struct be_ctx *ctx, const char *service_name,
user_data, primary);
if (ret && ret != EEXIST) {
DEBUG(SSSDBG_CRIT_FAILURE,
- "Failed to add server to failover service\n");
+ "Failed to add server to failover service [%d]: %s\n",
+ ret, sss_strerror(ret));
return ret;
}
@@ -465,7 +468,9 @@ static void be_primary_server_done(struct tevent_req *subreq)
ret = be_primary_server_timeout_activate(ctx->bctx, ctx->ev, ctx->bctx,
ctx->svc, ctx->timeout);
if (ret != EOK) {
- DEBUG(SSSDBG_MINOR_FAILURE, "Could not schedule primary server lookup\n");
+ DEBUG(SSSDBG_MINOR_FAILURE,
+ "Could not schedule primary server lookup [%d]: %s\n",
+ ret, sss_strerror(ret));
}
} else if (ret == EOK) {
be_run_reconnect_cb(ctx->bctx);
@@ -593,7 +598,8 @@ static void be_resolve_server_done(struct tevent_req *subreq)
return;
fail:
- DEBUG(SSSDBG_TRACE_LIBS, "Server resolution failed: %d\n", ret);
+ DEBUG(SSSDBG_TRACE_LIBS,
+ "Server resolution failed: [%d]: %s\n", ret, sss_strerror(ret));
state->svc->first_resolved = NULL;
tevent_req_error(req, ret);
}
@@ -625,8 +631,8 @@ errno_t be_resolve_server_process(struct tevent_req *subreq,
return EFAULT;
}
DEBUG(SSSDBG_MINOR_FAILURE,
- "Couldn't resolve server (%s), resolver returned (%d)\n",
- fo_get_server_str_name(state->srv), ret);
+ "Couldn't resolve server (%s), resolver returned [%d]: %s\n",
+ fo_get_server_str_name(state->srv), ret, sss_strerror(ret));
state->attempts++;
if (state->attempts >= 10) {