diff options
| author | Ravishankar N <ravishankar@redhat.com> | 2016-03-25 18:48:30 +0530 | 
|---|---|---|
| committer | Pranith Kumar Karampuri <pkarampu@redhat.com> | 2016-04-04 10:33:56 -0700 | 
| commit | 76ed61bb5e3fefb3e4071a24f6dd26f37c950c77 (patch) | |
| tree | 3a6b4daf719788794238b0e2754822fe76635f75 /cli | |
| parent | 028afb21a7793d3efbb9db431bde37ec332d9839 (diff) | |
afr: add mtime based split-brain resolution to CLI
Backport of http://review.gluster.org/#/c/13828/
Extended the CLI to include support for split-brain resolution based on
mtime. The command syntax is:
$:gluster volume heal <VOLNAME> split-brain latest-mtime <FILE>
where <FILE> can be either the full file name as seen from the root of the
volume (or) the gfid-string representation of the file.
Change-Id: I7a16f72ff1a4495aa69f43f22758a9404e958b4f
BUG: 1321748
Signed-off-by: Ravishankar N <ravishankar@redhat.com>
Reviewed-on: http://review.gluster.org/13838
Smoke: Gluster Build System <jenkins@build.gluster.com>
NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org>
Reviewed-by: Anuradha Talur <atalur@redhat.com>
CentOS-regression: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
Diffstat (limited to 'cli')
| -rw-r--r-- | cli/src/cli-cmd-parser.c | 10 | ||||
| -rw-r--r-- | cli/src/cli-cmd-volume.c | 7 | ||||
| -rw-r--r-- | cli/src/cli-rpc-ops.c | 3 | 
3 files changed, 18 insertions, 2 deletions
diff --git a/cli/src/cli-cmd-parser.c b/cli/src/cli-cmd-parser.c index 8fb78230935..206acddaa83 100644 --- a/cli/src/cli-cmd-parser.c +++ b/cli/src/cli-cmd-parser.c @@ -3604,6 +3604,16 @@ cli_cmd_volume_heal_options_parse (const char **words, int wordcount,                                  goto out;                          goto done;                  } +                if (!strcmp (words[4], "latest-mtime")) { +                        ret = dict_set_int32 (dict, "heal-op", +                                       GF_SHD_OP_SBRAIN_HEAL_FROM_LATEST_MTIME); +                        if (ret) +                                goto out; +                        ret = dict_set_str (dict, "file", (char *)words[5]); +                        if (ret) +                                goto out; +                        goto done; +                }                  if (!strcmp (words[4], "source-brick")) {                          ret = dict_set_int32 (dict, "heal-op",                                                GF_SHD_OP_SBRAIN_HEAL_FROM_BRICK); diff --git a/cli/src/cli-cmd-volume.c b/cli/src/cli-cmd-volume.c index f5926588a9e..26936c2a323 100644 --- a/cli/src/cli-cmd-volume.c +++ b/cli/src/cli-cmd-volume.c @@ -2115,6 +2115,7 @@ cli_print_brick_status (cli_volume_status_t *status)  }  #define NEEDS_GLFS_HEAL(op) ((op == GF_SHD_OP_SBRAIN_HEAL_FROM_BIGGER_FILE) || \ +                             (op == GF_SHD_OP_SBRAIN_HEAL_FROM_LATEST_MTIME) ||\                               (op == GF_SHD_OP_SBRAIN_HEAL_FROM_BRICK) ||      \                               (op == GF_SHD_OP_INDEX_SUMMARY) ||               \                               (op == GF_SHD_OP_SPLIT_BRAIN_FILES)) @@ -2143,6 +2144,10 @@ cli_launch_glfs_heal (int heal_op, dict_t *options)                  ret = dict_get_str (options, "file", &filename);                  runner_add_args (&runner, "bigger-file", filename, NULL);                  break; +        case GF_SHD_OP_SBRAIN_HEAL_FROM_LATEST_MTIME: +                ret = dict_get_str (options, "file", &filename); +                runner_add_args (&runner, "latest-mtime", filename, NULL); +                break;          case GF_SHD_OP_SBRAIN_HEAL_FROM_BRICK:                  ret = dict_get_str (options, "heal-source-hostname",                                      &hostname); @@ -2627,7 +2632,7 @@ struct cli_cmd volume_cmds[] = {          { "volume heal <VOLNAME> [enable | disable | full |"            "statistics [heal-count [replica <HOSTNAME:BRICKNAME>]] |"            "info [healed | heal-failed | split-brain] |" -          "split-brain {bigger-file <FILE> |" +          "split-brain {bigger-file <FILE> | latest-mtime <FILE> |"                         "source-brick <HOSTNAME:BRICKNAME> [<FILE>]}]",            cli_cmd_volume_heal_cbk,            "self-heal commands on volume specified by <VOLNAME>"}, diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index 275eb545980..9b2699a6471 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -8469,9 +8469,10 @@ gf_cli_heal_volume_cbk (struct rpc_req *req, struct iovec *iov,          case    GF_SHD_OP_STATISTICS_HEAL_COUNT_PER_REPLICA:                  heal_op_str = "count of entries to be healed per replica";                  break; -        /* The below 2 cases are never hit; they're coded only to make +        /* The below 3 cases are never hit; they're coded only to make           * compiler warnings go away.*/          case    GF_SHD_OP_SBRAIN_HEAL_FROM_BIGGER_FILE: +        case    GF_SHD_OP_SBRAIN_HEAL_FROM_LATEST_MTIME:          case    GF_SHD_OP_SBRAIN_HEAL_FROM_BRICK:                  break;  | 
