a3470f
From 717fab7db9afb3d2f512fcdb7a952eb57b67e8dd Mon Sep 17 00:00:00 2001
a3470f
From: Mohit Agrawal <moagrawa@redhat.com>
a3470f
Date: Thu, 10 May 2018 16:54:44 +0530
a3470f
Subject: [PATCH 268/271] dht: Excessive 'dict is null' logs in
a3470f
 dht_discover_complete
a3470f
a3470f
Problem: In Geo-Rep setup excessive "dict is null" logs in
a3470f
         dht_discover_complete while xattr is NULL
a3470f
a3470f
Solution: To avoid the logs update a condition in dht_discover_complete
a3470f
a3470f
> BUG: 1576767
a3470f
> Change-Id: Ic7aad712d9b6d69b85b76e4fdf2881adb0512237
a3470f
> fixes: bz#1576767
a3470f
> (cherry pick from commit 08dcb1c2bf02bba313d3bfc8fbc1c8ea63296321)
a3470f
> (Upstream review link https://review.gluster.org/#/c/20001/)
a3470f
a3470f
BUG: 1565577
a3470f
Change-Id: I1b1327eeef05fa261c814069a3bdd75653b37bd1
a3470f
Signed-off-by: Mohit Agrawal <moagrawa@redhat.com>
a3470f
Reviewed-on: https://code.engineering.redhat.com/gerrit/138490
a3470f
Tested-by: RHGS Build Bot <nigelb@redhat.com>
a3470f
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
a3470f
---
a3470f
 xlators/cluster/dht/src/dht-common.c | 3 ++-
a3470f
 1 file changed, 2 insertions(+), 1 deletion(-)
a3470f
a3470f
diff --git a/xlators/cluster/dht/src/dht-common.c b/xlators/cluster/dht/src/dht-common.c
a3470f
index ec1628a..feeaa3f 100644
a3470f
--- a/xlators/cluster/dht/src/dht-common.c
a3470f
+++ b/xlators/cluster/dht/src/dht-common.c
a3470f
@@ -748,7 +748,8 @@ cleanup:
a3470f
 done:
a3470f
         dht_set_fixed_dir_stat (&local->postparent);
a3470f
         /* Delete mds xattr at the time of STACK UNWIND */
a3470f
-        GF_REMOVE_INTERNAL_XATTR (conf->mds_xattr_key, local->xattr);
a3470f
+        if (local->xattr)
a3470f
+                GF_REMOVE_INTERNAL_XATTR (conf->mds_xattr_key, local->xattr);
a3470f
 
a3470f
         DHT_STACK_UNWIND (lookup, main_frame, local->op_ret, local->op_errno,
a3470f
                           local->inode, &local->stbuf, local->xattr,
a3470f
-- 
a3470f
1.8.3.1
a3470f