glusterd/snapshot: Added trace statements and handled snapshot create commit failure.

Also handles empty string(not NULL) in gd_syncop_mgmt_brick_op()
and adds "Snapshot" in operation list used for printing op
during logging.

Change-Id: Icac9dce6bf1c087ab2aace9953e2af3a0fb81be6
Signed-off-by: Avra Sengupta <asengupt@redhat.com>

1 files changed, 10 insertions, 10 deletions

diff --git a/xlators/mgmt/glusterd/src/glusterd-mgmt-handler.c b/xlators/mgmt/glusterd/src/glusterd-mgmt-handler.c
index 63bbc6687..236d9b72f 100644
--- a/xlators/mgmt/glusterd/src/glusterd-mgmt-handler.c
+++ b/xlators/mgmt/glusterd/src/glusterd-mgmt-handler.c
@@ -88,7 +88,7 @@ glusterd_syctasked_volume_lock (rpcsvc_request_t *req,
 out:
         glusterd_mgmt_v3_vol_lock_send_resp (req, ret);
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }
 
@@ -126,7 +126,7 @@ out:
         glusterd_friend_sm ();
         glusterd_op_sm ();
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }
 
@@ -202,7 +202,7 @@ out:
                         GF_FREE (ctx);
         }
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }
 
@@ -326,7 +326,7 @@ out:
         if (rsp_dict)
                 dict_unref (rsp_dict);
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }
 
@@ -450,7 +450,7 @@ out:
         if (rsp_dict)
                 dict_unref (rsp_dict);
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }
 
@@ -573,7 +573,7 @@ out:
         if (rsp_dict)
                 dict_unref (rsp_dict);
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }
 
@@ -698,7 +698,7 @@ out:
         if (rsp_dict)
                 dict_unref (rsp_dict);
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }
 
@@ -764,7 +764,7 @@ glusterd_syctasked_volume_unlock (rpcsvc_request_t *req,
 out:
         glusterd_mgmt_v3_vol_unlock_send_resp (req, ret);
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }
 
@@ -790,7 +790,7 @@ glusterd_op_state_machine_volume_unlock (rpcsvc_request_t *req,
         glusterd_friend_sm ();
         glusterd_op_sm ();
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }
 
@@ -866,7 +866,7 @@ out:
                         GF_FREE (ctx);
         }
 
-        gf_log (this->name, GF_LOG_DEBUG, "Returning %d", ret);
+        gf_log (this->name, GF_LOG_TRACE, "Returning %d", ret);
         return ret;
 }