diff options
| author | Anand Avati <avati@redhat.com> | 2012-05-29 22:30:53 -0700 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-05-29 22:58:24 -0700 |
| commit | 493ef71222e780cb542c7edc5dfb7d0e3097c55a (patch) | |
| tree | dd9adc6c5ebe1250233d89e7bf6f19f20abd4d2d | |
| parent | f69785a9e3f9ad55e81f1fe8212485b7e4dc11fe (diff) | |
protocol: do not log getxattr/ENODATA as warningv3.3.0qa45
When SELinux is enabled, most of the files do not have labels and
result is a ton of unnecessary logs
Change-Id: I0e781e2fb6bcfb3fb12298175a41f7b981af9c39
BUG: 811217
Signed-off-by: Anand Avati <avati@redhat.com>
Reviewed-on: http://review.gluster.com/3487
Tested-by: Gluster Build System <jenkins@build.gluster.com>
| -rw-r--r-- | xlators/protocol/client/src/client3_1-fops.c | 2 | ||||
| -rw-r--r-- | xlators/protocol/server/src/server3_1-fops.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/xlators/protocol/client/src/client3_1-fops.c b/xlators/protocol/client/src/client3_1-fops.c index c4164a95b..224a26dc7 100644 --- a/xlators/protocol/client/src/client3_1-fops.c +++ b/xlators/protocol/client/src/client3_1-fops.c @@ -1049,7 +1049,7 @@ client3_1_getxattr_cbk (struct rpc_req *req, struct iovec *iov, int count, out: if (rsp.op_ret == -1) { - gf_log (this->name, ((op_errno == ENOTSUP) ? + gf_log (this->name, (((op_errno == ENOTSUP) || (op_errno == ENODATA)) ? GF_LOG_DEBUG : GF_LOG_WARNING), "remote operation failed: %s. Path: %s (%s). Key: %s", strerror (op_errno), diff --git a/xlators/protocol/server/src/server3_1-fops.c b/xlators/protocol/server/src/server3_1-fops.c index 2e0bbb4c8..1169da216 100644 --- a/xlators/protocol/server/src/server3_1-fops.c +++ b/xlators/protocol/server/src/server3_1-fops.c @@ -816,7 +816,7 @@ out: rsp.op_errno = gf_errno_to_error (op_errno); if (op_ret == -1) - gf_log (this->name, ((op_errno == ENOTSUP) ? + gf_log (this->name, (((op_errno == ENOTSUP) || (op_errno == ENODATA)) ? GF_LOG_DEBUG : GF_LOG_INFO), "%"PRId64": GETXATTR %s (%s) ==> %"PRId32" (%s)", frame->root->unique, state->loc.path, |