diff options
| author | Rajesh Joseph <rjoseph@redhat.com> | 2016-05-24 15:10:33 +0530 |
|---|---|---|
| committer | Jeff Darcy <jdarcy@redhat.com> | 2016-05-25 07:22:55 -0700 |
| commit | dca1c08c6384e34403d8b5ad6eaf0931529168ac (patch) | |
| tree | ae4edd7885d77a73856a00cdd5bcc699bde8cf7b | |
| parent | ce466fa6353394a6ac4ca9922237de77686d64d0 (diff) | |
snapshot/uss: log is filled with gfid is NULL error
In snapview-server if the lookup came on missing entry
then it does not have proper context to see if it is the
case of "Stale file handle" or ENOENT. We should not log
this error in GF_LOG_ERROR level, instead this should be
a debug log.
Change-Id: I8f7cc0e117e9a6dd9482da2e94a56f2ed731a442
BUG: 1339149
Signed-off-by: Rajesh Joseph <rjoseph@redhat.com>
Reviewed-on: http://review.gluster.org/14514
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Jeff Darcy <jdarcy@redhat.com>
| -rw-r--r-- | xlators/features/snapview-server/src/snapview-server.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/xlators/features/snapview-server/src/snapview-server.c b/xlators/features/snapview-server/src/snapview-server.c index 338d5bd2f75..72cfc908bba 100644 --- a/xlators/features/snapview-server/src/snapview-server.c +++ b/xlators/features/snapview-server/src/snapview-server.c @@ -612,7 +612,9 @@ svs_lookup (call_frame_t *frame, xlator_t *this, loc_t *loc, dict_t *xdata) if (!inode_ctx && !parent_ctx) { if (gf_uuid_is_null (loc->gfid) && gf_uuid_is_null (loc->inode->gfid)) { - gf_log (this->name, GF_LOG_ERROR, "gfid is NULL"); + gf_log (this->name, GF_LOG_DEBUG, "gfid is NULL, " + "either the lookup came on missing entry or " + "the entry is stale"); op_ret = -1; op_errno = ESTALE; goto out; |