summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKaushal M <kaushal@redhat.com>2012-12-10 11:56:41 +0530
committerAnand Avati <avati@redhat.com>2012-12-10 14:09:15 -0800
commit07c40a2d149f9adfa4ce400051c1313127349192 (patch)
treec1b1449550226128ac29182c4004765db5efae2e
parentc399ca2fe3b81e19b9ff7ddd3f8bebe41179be48 (diff)
rpcsvc: Add an extra log message
Adds an extra log message in rpcvsc_check_and_reply_error(). Also, makes a small change in the test-script. Change-Id: I2b686e6fa86529cc4fbf0066df64057a9784b849 BUG: 884452 Signed-off-by: Kaushal M <kaushal@redhat.com> Reviewed-on: http://review.gluster.org/4285 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Anand Avati <avati@redhat.com>
-rw-r--r--rpc/rpc-lib/src/rpcsvc.c4
-rw-r--r--tests/bugs/bug-884452.t4
2 files changed, 6 insertions, 2 deletions
diff --git a/rpc/rpc-lib/src/rpcsvc.c b/rpc/rpc-lib/src/rpcsvc.c
index eef1f05044c..1c49c09b1f4 100644
--- a/rpc/rpc-lib/src/rpcsvc.c
+++ b/rpc/rpc-lib/src/rpcsvc.c
@@ -443,6 +443,10 @@ rpcsvc_check_and_reply_error (int ret, call_frame_t *frame, void *opaque)
req = opaque;
+ if (ret)
+ gf_log ("rpcsvc", GF_LOG_ERROR,
+ "rpc actor failed to complete successfully");
+
if (ret == RPCSVC_ACTOR_ERROR) {
ret = rpcsvc_error_reply (req);
if (ret)
diff --git a/tests/bugs/bug-884452.t b/tests/bugs/bug-884452.t
index 420b4bc8847..27d7e6afcce 100644
--- a/tests/bugs/bug-884452.t
+++ b/tests/bugs/bug-884452.t
@@ -36,8 +36,8 @@ function vol-status-loop
TEST vol-status-loop
-kill $LS_LOOP >/dev/null 2>&1
-sleep 2
+kill -KILL $LS_LOOP >/dev/null 2>&1
+sleep 2;
TEST umount $M0