From: MORITA Kazutaka <morita.kazutaka at lab.ntt.co.jp> If cpg_initialize() returns CS_ERR_SECURITY, the error reason is sheep doesn't have a privilege to use corosync because of corosync configuration. In this case, printing "Permission denied" is better. In the other cases, corosync is actually running and there is another error reason. This removes "is corosync running?" from the error message in the default cases. Signed-off-by: MORITA Kazutaka <morita.kazutaka at lab.ntt.co.jp> --- sheep/cluster/corosync.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/sheep/cluster/corosync.c b/sheep/cluster/corosync.c index 0bd3eb5..7166c11 100644 --- a/sheep/cluster/corosync.c +++ b/sheep/cluster/corosync.c @@ -735,9 +735,11 @@ again: sd_debug("retry cpg_initialize"); usleep(200000); goto again; + case CS_ERR_SECURITY: + sd_err("failed to initialize cpg - permission denied"); + return -1; default: - sd_err("failed to initialize cpg (%d) - is corosync running?", - ret); + sd_err("failed to initialize cpg (%d)", ret); return -1; } -- 1.7.9.5 |