[sheepdog] [PATCH] sheep: changed end of auto-recovery message to notice

Hitoshi Mitake mitake.hitoshi at gmail.com
Thu Feb 6 09:27:41 CET 2014


At Thu,  6 Feb 2014 17:09:42 +0900,
Yoshinori Matsuo wrote:
> 
> Sheep output the end of auto-recovery log message (all nodes are recovered)
> at DEBUG level.
> When sheep nodes are working in the INFO level or upper,
> the operator could not get this message
> even if operators want to know the completion of auto-recovery.
> 
> This patch raised the log message to the NOTICE level.
> 
> Signed-off-by: Yoshinori Matsuo <matsuo.yoshinori at lab.ntt.co.jp>
> ---
>  sheep/ops.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Looks good to me.
Reviewed-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>

> 
> diff --git a/sheep/ops.c b/sheep/ops.c
> index 481789f..bd663eb 100644
> --- a/sheep/ops.c
> +++ b/sheep/ops.c
> @@ -695,7 +695,7 @@ static int cluster_recovery_completion(const struct sd_req *req,
>  				break;
>  		}
>  		if (i == nr_recovereds) {
> -			sd_debug("all nodes are recovered, epoch %d", epoch);
> +			sd_notice("all nodes are recovered, epoch %d", epoch);
>  			last_gathered_epoch = epoch;
>  			/* sd_store can be NULL if this node is a gateway */
>  			if (vnode_info->nr_zones >= ec_max_data_strip &&
> -- 
> 1.7.2.5
> 
> -- 
> sheepdog mailing list
> sheepdog at lists.wpkg.org
> http://lists.wpkg.org/mailman/listinfo/sheepdog



More information about the sheepdog mailing list