From ed430fc04e57c89d08cfdd1bb5e408c5baf53adf Mon Sep 17 00:00:00 2001 From: ankit Date: Tue, 30 Aug 2016 12:55:32 +0530 Subject: dht: Proper log message if data migration is skipped Change-Id: Id0af15a2aec96bdbe675b4c959b56f0fc8e72504 BUG: 1341948 Signed-off-by: ankit Reviewed-on: http://review.gluster.org/15345 Tested-by: ankitraj Smoke: Gluster Build System NetBSD-regression: NetBSD Build System Reviewed-by: N Balachandran CentOS-regression: Gluster Build System --- xlators/cluster/dht/src/dht-rebalance.c | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c index f7fa267b7f5..6d93cf24d1d 100644 --- a/xlators/cluster/dht/src/dht-rebalance.c +++ b/xlators/cluster/dht/src/dht-rebalance.c @@ -715,14 +715,14 @@ __dht_check_free_space (xlator_t *to, xlator_t *from, loc_t *loc, (src_statfs_blocks + stbuf->ia_blocks)) { gf_msg (this->name, GF_LOG_WARNING, 0, - DHT_MSG_MIGRATE_FILE_FAILED, - "data movement attempted from node " - "(%s:%"PRIu64") with higher disk space " - "to a node (%s:%"PRIu64") with lesser " - "disk space, file { blocks:%"PRIu64", " - "name:(%s) }", from->name, src_statfs_blocks, - to->name, dst_statfs_blocks, - stbuf->ia_blocks, loc->path); + DHT_MSG_MIGRATE_FILE_FAILED, + "data movement of file " + "{blocks:%"PRIu64" name:(%s) } would result in " + "dst node (%s:%"PRIu64") having lower disk " + "space then the source node (%s:%"PRIu64")" + ".Skipping file.", stbuf->ia_blocks, loc->path, + to->name, dst_statfs_blocks, from->name, + src_statfs_blocks); /* this is not a 'failure', but we don't want to consider this as 'success' too :-/ */ -- cgit