glusterd: glusterd is releasing the locks before timeout

Problem: We introduced lock timer in mgmt v3, which will realease
the lock after 3 minutes from command execution. Some commands related
to heal/profile will take more time to execute. For these comands
timeout is set to 10 minutes. As the lock timer is set to 3 minutes
glusterd is releasing the lock after 3 minutes. That means locks are
released before the command is completed its execution.

Solution: Pass a timeout parameter from cli to glusterd, when there
is a change in default timeout value(i.e, default timeout value can
be changed through command line or for the commands related to profile/heal
we will change the default timeout value to 10 minutes.) glusterd will
set the lock timer timeout according to the timeout value passed.

Change-Id: I7b7a9a4f95ed44aca39ef9d9907f546bca99c69d
fixes: bz#1577731
Signed-off-by: Sanju Rakonde <srakonde@redhat.com>

6 files changed, 58 insertions, 0 deletions

diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c
index 71e66d5056c..323b8340d87 100644
--- a/xlators/mgmt/glusterd/src/glusterd-handler.c
+++ b/xlators/mgmt/glusterd/src/glusterd-handler.c
@@ -718,6 +718,7 @@ glusterd_op_txn_begin (rpcsvc_request_t *req, glusterd_op_t op, void *ctx,
         glusterd_op_info_t          txn_op_info     = {{0},};
         glusterd_op_sm_event_type_t event_type      = GD_OP_EVENT_NONE;
         uint32_t                    op_errno        = 0;
+        uint32_t                    timeout          = 0;
 
         GF_ASSERT (req);
         GF_ASSERT ((op > GD_OP_NONE) && (op < GD_OP_MAX));
@@ -785,6 +786,15 @@ glusterd_op_txn_begin (rpcsvc_request_t *req, glusterd_op_t op, void *ctx,
                                 goto out;
                 }
 
+                /* Cli will add timeout key to dict if the default timeout is
+                 * other than 2 minutes. Here we use this value to check whether
+                 * mgmt_v3_lock_timeout should be set to default value or we
+                 * need to change the value according to timeout value
+                 * i.e, timeout + 120 seconds. */
+                ret = dict_get_uint32 (dict, "timeout", &timeout);
+                if (!ret)
+                        priv->mgmt_v3_lock_timeout = timeout + 120;
+
                 ret = glusterd_mgmt_v3_lock (volname, MY_UUID, &op_errno,
                                              "vol");
                 if (ret) {
diff --git a/xlators/mgmt/glusterd/src/glusterd-locks.c b/xlators/mgmt/glusterd/src/glusterd-locks.c
index 33277058db8..ab69c78b0f0 100644
--- a/xlators/mgmt/glusterd/src/glusterd-locks.c
+++ b/xlators/mgmt/glusterd/src/glusterd-locks.c
@@ -656,6 +656,8 @@ glusterd_mgmt_v3_lock (const char *name, uuid_t uuid, uint32_t *op_errno,
         key_dup = gf_strdup (key);
         delay.tv_sec = priv->mgmt_v3_lock_timeout;
         delay.tv_nsec = 0;
+        /*changing to default timeout value*/
+        priv->mgmt_v3_lock_timeout = GF_LOCK_TIMER;
 
         ret = -1;
         mgmt_lock_timer_xl = mgmt_lock_timer->xl;
diff --git a/xlators/mgmt/glusterd/src/glusterd-mgmt-handler.c b/xlators/mgmt/glusterd/src/glusterd-mgmt-handler.c
index 5b7f0fa3c25..993f12af103 100644
--- a/xlators/mgmt/glusterd/src/glusterd-mgmt-handler.c
+++ b/xlators/mgmt/glusterd/src/glusterd-mgmt-handler.c
@@ -132,8 +132,12 @@ glusterd_handle_mgmt_v3_lock_fn (rpcsvc_request_t *req)
         xlator_t               *this            = NULL;
         gf_boolean_t            is_synctasked   = _gf_false;
         gf_boolean_t            free_ctx        = _gf_false;
+        glusterd_conf_t        *conf            = NULL;
+        uint32_t                timeout         = 0;
 
         this = THIS;
+        conf = this->private;
+        GF_ASSERT (conf);
         GF_ASSERT (this);
         GF_ASSERT (req);
 
@@ -183,6 +187,15 @@ glusterd_handle_mgmt_v3_lock_fn (rpcsvc_request_t *req)
                 goto out;
         }
 
+        /* Cli will add timeout key to dict if the default timeout is
+         * other than 2 minutes. Here we use this value to check whether
+         * mgmt_v3_lock_timeout should be set to default value or we
+         * need to change the value according to timeout value
+         * i.e, timeout + 120 seconds. */
+        ret = dict_get_uint32 (ctx->dict, "timeout", &timeout);
+        if (!ret)
+                conf->mgmt_v3_lock_timeout = timeout + 120;
+
         is_synctasked = dict_get_str_boolean (ctx->dict,
                                               "is_synctasked", _gf_false);
         if (is_synctasked) {
diff --git a/xlators/mgmt/glusterd/src/glusterd-mgmt.c b/xlators/mgmt/glusterd/src/glusterd-mgmt.c
index 19eefab6f3c..30891a006b6 100644
--- a/xlators/mgmt/glusterd/src/glusterd-mgmt.c
+++ b/xlators/mgmt/glusterd/src/glusterd-mgmt.c
@@ -740,6 +740,7 @@ glusterd_mgmt_v3_initiate_lockdown (glusterd_op_t op, dict_t *dict,
         uuid_t               peer_uuid  = {0};
         xlator_t            *this       = NULL;
         glusterd_conf_t     *conf       = NULL;
+        uint32_t             timeout    = 0;
 
         this = THIS;
         GF_ASSERT (this);
@@ -750,6 +751,15 @@ glusterd_mgmt_v3_initiate_lockdown (glusterd_op_t op, dict_t *dict,
         GF_ASSERT (op_errstr);
         GF_ASSERT (is_acquired);
 
+        /* Cli will add timeout key to dict if the default timeout is
+         * other than 2 minutes. Here we use this value to check whether
+         * mgmt_v3_lock_timeout should be set to default value or we
+         * need to change the value according to timeout value
+         * i.e, timeout + 120 seconds. */
+        ret = dict_get_uint32 (dict, "timeout", &timeout);
+        if (!ret)
+                conf->mgmt_v3_lock_timeout = timeout + 120;
+
         /* Trying to acquire multiple mgmt_v3 locks on local node */
         ret = glusterd_multiple_mgmt_v3_lock (dict, MY_UUID, op_errno);
         if (ret) {
diff --git a/xlators/mgmt/glusterd/src/glusterd-op-sm.c b/xlators/mgmt/glusterd/src/glusterd-op-sm.c
index 64855cd60fa..956cd256c4f 100644
--- a/xlators/mgmt/glusterd/src/glusterd-op-sm.c
+++ b/xlators/mgmt/glusterd/src/glusterd-op-sm.c
@@ -4143,11 +4143,16 @@ glusterd_op_ac_lock (glusterd_op_sm_event_t *event, void *ctx)
         glusterd_op_lock_ctx_t         *lock_ctx        = NULL;
         xlator_t                       *this            = NULL;
         uint32_t                        op_errno        = 0;
+        glusterd_conf_t                *conf            = NULL;
+        uint32_t                        timeout         = 0;
 
         GF_ASSERT (event);
         GF_ASSERT (ctx);
 
         this = THIS;
+        GF_ASSERT (this);
+        conf = this->private;
+        GF_ASSERT (conf);
 
         lock_ctx = (glusterd_op_lock_ctx_t *)ctx;
 
@@ -4158,6 +4163,15 @@ glusterd_op_ac_lock (glusterd_op_sm_event_t *event, void *ctx)
                 ret = glusterd_lock (lock_ctx->uuid);
                 glusterd_op_lock_send_resp (lock_ctx->req, ret);
         } else {
+                /* Cli will add timeout key to dict if the default timeout is
+                 * other than 2 minutes. Here we use this value to check whether
+                 * mgmt_v3_lock_timeout should be set to default value or we
+                 * need to change the value according to timeout value
+                 * i.e, timeout + 120 seconds. */
+                ret = dict_get_uint32 (lock_ctx->dict, "timeout", &timeout);
+                if (!ret)
+                        conf->mgmt_v3_lock_timeout = timeout + 120;
+
                 ret = dict_get_str (lock_ctx->dict, "volname", &volname);
                 if (ret)
                         gf_msg (this->name, GF_LOG_ERROR, 0,
diff --git a/xlators/mgmt/glusterd/src/glusterd-syncop.c b/xlators/mgmt/glusterd/src/glusterd-syncop.c
index e5d4421deb4..d00a3d0f152 100644
--- a/xlators/mgmt/glusterd/src/glusterd-syncop.c
+++ b/xlators/mgmt/glusterd/src/glusterd-syncop.c
@@ -1809,6 +1809,7 @@ gd_sync_task_begin (dict_t *op_ctx, rpcsvc_request_t * req)
         glusterd_op_info_t          txn_opinfo       = {{0},};
         uint32_t                    op_errno         = 0;
         gf_boolean_t                cluster_lock     = _gf_false;
+        uint32_t                    timeout          = 0;
 
         this = THIS;
         GF_ASSERT (this);
@@ -1870,6 +1871,14 @@ gd_sync_task_begin (dict_t *op_ctx, rpcsvc_request_t * req)
                         goto out;
                 }
         } else {
+                /* Cli will add timeout key to dict if the default timeout is
+                 * other than 2 minutes. Here we use this value to check whether
+                 * mgmt_v3_lock_timeout should be set to default value or we
+                 * need to change the value according to timeout value
+                 * i.e, timeout + 120 seconds. */
+                ret = dict_get_uint32 (op_ctx, "timeout", &timeout);
+                if (!ret)
+                        conf->mgmt_v3_lock_timeout = timeout + 120;
 
                 ret = dict_get_str (op_ctx, "globalname", &global);
                 if (!ret) {