diff options
| author | Niels de Vos <ndevos@redhat.com> | 2015-07-25 10:47:03 +0200 |
|---|---|---|
| committer | Niels de Vos <ndevos@redhat.com> | 2015-07-26 01:10:24 -0700 |
| commit | 285d1c851d18bcd7e7a2c48f440bbe107d260dca (patch) | |
| tree | 385481655c9078f1cf0fcfe8f221b4e21a76ec16 /xlators | |
| parent | c12234ee25deafe148a1557c30a0e56688548f91 (diff) | |
logging: client3_3_removexattr_cbk should not log expected ENODATA
Commit 379dbbfd changed the log level of an expected error message from
GF_LOG_DEBUG to '0'. This causes the message to always get logged. It is
intended to log the warning when the xattr could not be removed (but
exists), removing a non-existing xattr is sufficient as debug.
Backported from:
> BUG: 1246736
> Change-Id: If11d15d699e8152cc32e0c0c1d65b587c5c48892
> Reviewed-on: http://review.gluster.org/11759
> Signed-off-by: Niels de Vos <ndevos@redhat.com>
Change-Id: If11d15d699e8152cc32e0c0c1d65b587c5c48892
BUG: 1246728
Signed-off-by: Niels de Vos <ndevos@redhat.com>
Reviewed-on: http://review.gluster.org/11760
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Joe Julian <me@joejulian.name>
Tested-by: Joe Julian <me@joejulian.name>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Diffstat (limited to 'xlators')
| -rw-r--r-- | xlators/protocol/client/src/client-rpc-fops.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xlators/protocol/client/src/client-rpc-fops.c b/xlators/protocol/client/src/client-rpc-fops.c index 9512087e413..2c0b477f2c8 100644 --- a/xlators/protocol/client/src/client-rpc-fops.c +++ b/xlators/protocol/client/src/client-rpc-fops.c @@ -1288,7 +1288,7 @@ client3_3_removexattr_cbk (struct rpc_req *req, struct iovec *iov, int count, out: if (rsp.op_ret == -1) { if ((ENODATA == rsp.op_errno) || (ENOATTR == rsp.op_errno)) - loglevel = 0; + loglevel = GF_LOG_DEBUG; else loglevel = GF_LOG_WARNING; |