diff options
| author | Ravishankar N <ravishankar@redhat.com> | 2016-10-04 10:10:56 +0530 |
|---|---|---|
| committer | Vijay Bellur <vbellur@redhat.com> | 2016-10-04 11:32:05 -0700 |
| commit | a1bbbf35cb91e5132b8c46dff0681d24371295a4 (patch) | |
| tree | 90f90cb992fa0b910d3151ffd49604ba11085f66 | |
| parent | 9ab5b52dee5be45458fdb5446d3cbf6a1a5306a6 (diff) | |
afr: fix incorrect debug log in selfheal path
1. While looking at glustershd logs in DEBUG log-level, it was found that
all bricks of the replica were printed as local bricks even though they
were not. Fixed it.
2. Print the name of the subvol from which the entry was got during
index crawl.
Change-Id: I08b32e38694c755715e9fe0c0e1dd9212abcfb16
BUG: 1381421
Signed-off-by: Ravishankar N <ravishankar@redhat.com>
Reviewed-on: http://review.gluster.org/15610
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Smoke: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>
| -rw-r--r-- | libglusterfs/src/syncop-utils.c | 2 | ||||
| -rw-r--r-- | xlators/cluster/afr/src/afr-self-heald.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/libglusterfs/src/syncop-utils.c b/libglusterfs/src/syncop-utils.c index 8f25db237f5..029f10b058e 100644 --- a/libglusterfs/src/syncop-utils.c +++ b/libglusterfs/src/syncop-utils.c @@ -542,7 +542,7 @@ syncop_is_subvol_local (xlator_t *this, loc_t *loc, gf_boolean_t *is_local) ret = glusterfs_is_local_pathinfo (pathinfo, is_local); gf_msg_debug (this->name, 0, "subvol %s is %slocal", - this->name, is_local ? "" : "not "); + this->name, *is_local ? "" : "not "); out: if (xattr) diff --git a/xlators/cluster/afr/src/afr-self-heald.c b/xlators/cluster/afr/src/afr-self-heald.c index 7ccac919769..8a3a5521409 100644 --- a/xlators/cluster/afr/src/afr-self-heald.c +++ b/xlators/cluster/afr/src/afr-self-heald.c @@ -428,8 +428,8 @@ afr_shd_index_heal (xlator_t *subvol, gf_dirent_t *entry, loc_t *parent, if (!priv->shd.enabled) return -EBUSY; - gf_msg_debug (healer->this->name, 0, "got entry: %s", - entry->d_name); + gf_msg_debug (healer->this->name, 0, "got entry: %s from %s", + entry->d_name, priv->children[healer->subvol]->name); ret = gf_uuid_parse (entry->d_name, gfid); if (ret) |