|
 |
74096c |
From 1370db202a2a60810409f74c390448bf8fbd6998 Mon Sep 17 00:00:00 2001
|
|
|
74096c |
From: Barak Sason Rofman <bsasonro@redhat.com>
|
|
|
74096c |
Date: Sun, 9 Feb 2020 15:09:30 +0200
|
|
|
74096c |
Subject: [PATCH 429/449] dht/rebalance - fixing failure occurace due to
|
|
|
74096c |
rebalance stop
|
|
|
74096c |
|
|
|
74096c |
Probelm description:
|
|
|
74096c |
When topping rebalance, the following error messages appear in the
|
|
|
74096c |
rebalance log file:
|
|
|
74096c |
[2020-01-28 14:31:42.452070] W [dht-rebalance.c:3447:gf_defrag_process_dir] 0-distrep-dht: Found error from gf_defrag_get_entry
|
|
|
74096c |
[2020-01-28 14:31:42.452764] E [MSGID: 109111] [dht-rebalance.c:3971:gf_defrag_fix_layout] 0-distrep-dht: gf_defrag_process_dir failed for directory: /0/1/2/3/4/5/6/7/8/9/10/11/12/13/14/15/16/17/18/19/20/21/22/23/24/25/26/27/28/29/30/31
|
|
|
74096c |
[2020-01-28 14:31:42.453498] E [MSGID: 109016] [dht-rebalance.c:3906:gf_defrag_fix_layout] 0-distrep-dht: Fix layout failed for /0/1/2/3/4/5/6/7/8/9/10/11/12/13/14/15/16/17/18/19/20/21/22/23/24/25/26/27/28/29/30
|
|
|
74096c |
|
|
|
74096c |
In order to avoid seing these error messages, a modification to the
|
|
|
74096c |
error handling mechanism has been made.
|
|
|
74096c |
In addition, several log messages had been added in order to improve debugging efficiency
|
|
|
74096c |
|
|
|
74096c |
backport of https://review.gluster.org/#/c/glusterfs/+/24103/
|
|
|
74096c |
>fixes: bz#1800956
|
|
|
74096c |
>Change-Id: Ifc82dae79ab3da9fe22ee25088a2a6b855afcfcf
|
|
|
74096c |
>Signed-off-by: Barak Sason Rofman <bsasonro@redhat.com>
|
|
|
74096c |
|
|
|
74096c |
BUG: 1286171
|
|
|
74096c |
Change-Id: Ifc82dae79ab3da9fe22ee25088a2a6b855afcfcf
|
|
|
74096c |
Signed-off-by: Barak Sason Rofman <bsasonro@redhat.com>
|
|
|
74096c |
Reviewed-on: https://code.engineering.redhat.com/gerrit/202562
|
|
|
74096c |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
74096c |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
74096c |
---
|
|
|
74096c |
xlators/cluster/dht/src/dht-rebalance.c | 8 ++++++++
|
|
|
74096c |
1 file changed, 8 insertions(+)
|
|
|
74096c |
|
|
|
74096c |
diff --git a/xlators/cluster/dht/src/dht-rebalance.c b/xlators/cluster/dht/src/dht-rebalance.c
|
|
|
74096c |
index 8f31dca..88b6b54 100644
|
|
|
74096c |
--- a/xlators/cluster/dht/src/dht-rebalance.c
|
|
|
74096c |
+++ b/xlators/cluster/dht/src/dht-rebalance.c
|
|
|
74096c |
@@ -3479,6 +3479,10 @@ gf_defrag_process_dir(xlator_t *this, gf_defrag_info_t *defrag, loc_t *loc,
|
|
|
74096c |
migrate_data, dir_dfmeta, xattr_req,
|
|
|
74096c |
&should_commit_hash, perrno);
|
|
|
74096c |
|
|
|
74096c |
+ if (defrag->defrag_status == GF_DEFRAG_STATUS_STOPPED) {
|
|
|
74096c |
+ goto out;
|
|
|
74096c |
+ }
|
|
|
74096c |
+
|
|
|
74096c |
if (ret) {
|
|
|
74096c |
gf_log(this->name, GF_LOG_WARNING,
|
|
|
74096c |
"Found "
|
|
|
74096c |
@@ -3935,6 +3939,10 @@ gf_defrag_fix_layout(xlator_t *this, gf_defrag_info_t *defrag, loc_t *loc,
|
|
|
74096c |
ret = gf_defrag_fix_layout(this, defrag, &entry_loc, fix_layout,
|
|
|
74096c |
migrate_data);
|
|
|
74096c |
|
|
|
74096c |
+ if (defrag->defrag_status == GF_DEFRAG_STATUS_STOPPED) {
|
|
|
74096c |
+ goto out;
|
|
|
74096c |
+ }
|
|
|
74096c |
+
|
|
|
74096c |
if (ret && ret != 2) {
|
|
|
74096c |
gf_msg(this->name, GF_LOG_ERROR, 0, DHT_MSG_LAYOUT_FIX_FAILED,
|
|
|
74096c |
"Fix layout failed for %s", entry_loc.path);
|
|
|
74096c |
--
|
|
|
74096c |
1.8.3.1
|
|
|
74096c |
|