diff options
| author | hari gowtham <hgowtham@redhat.com> | 2015-12-04 18:34:36 +0530 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2015-12-15 19:45:42 -0800 |
| commit | 55f4e8a74e89d61c97e79474c4488ba0bf40a3c1 (patch) | |
| tree | e1e3e1d08c8741479163e8445cb311f744ae16c1 /cli/src | |
| parent | 6e17fb2097f941798e1d56728fd3d61e525a39a5 (diff) | |
tier/glusterd : making new tier detach command throw warning
For detach tier, the validation was done using the string "detach-tier"
but the new commands used has the string "tier". Making the string use
"tier" to compare, creates problem as the tier status and tier detach
have the keyword "tier". So tier detach and tier status were separated.
and strtok was used to prevent the condition from passing when the
volume name has a substring of "tier". (only the second word from the
string is got and checked if the feature is tier)
Problem: new detach tier command doesnt throw warnings like
"not a tier volume" or " detach tier not started" respectively
instead it prints empty output.
Fix: while validate the volume is checked if its a tiered volume
if yes it is checked if the detach tier is started, else a warning
is thrown respectively.
Change-Id: I94246d53b18ab0e9406beaf459eaddb7c5b766c2
BUG: 1288517
Signed-off-by: hari gowtham <hgowtham@redhat.com>
Reviewed-on: http://review.gluster.org/12883
Tested-by: NetBSD Build System <jenkins@build.gluster.org>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Diffstat (limited to 'cli/src')
| -rw-r--r-- | cli/src/cli-rpc-ops.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/cli/src/cli-rpc-ops.c b/cli/src/cli-rpc-ops.c index d59ff8b6d2c..666d44ff831 100644 --- a/cli/src/cli-rpc-ops.c +++ b/cli/src/cli-rpc-ops.c @@ -4539,7 +4539,7 @@ gf_cli_detach_tier (call_frame_t *frame, xlator_t *this, } else { /* Need rebalance status to be sent :-) */ if (command == GF_OP_CMD_STATUS) - cmd |= GF_DEFRAG_CMD_STATUS; + cmd |= GF_DEFRAG_CMD_DETACH_STATUS; else cmd |= GF_DEFRAG_CMD_STOP_DETACH_TIER; |