diff options
| author | Sachin Pandit <spandit@redhat.com> | 2014-06-09 07:52:37 +0530 |
|---|---|---|
| committer | Kaushal M <kaushal@redhat.com> | 2014-06-16 04:09:16 -0700 |
| commit | 452eea5c43ee5af450e91f240860551af6aba241 (patch) | |
| tree | 02d13e20d78b7bf1c995de364423b96753601c4c /xlators | |
| parent | 7342076f420efd62c9c46cdf5454b283225d1ec8 (diff) | |
glusterd/snapshot : override postvalidate for few snapshot commands.
snapshot info, list, config display and status does not
require any operations to be performed during postvalidate stage.
Hence it is better to override these commands in postvalidate.
Or else there will be a warning in log message saying "postvalidation
failed".
Change-Id: I14d64f7bf9adee8821067dd74d5027215d7b02f8
BUG: 1106406
Signed-off-by: Sachin Pandit <spandit@redhat.com>
Reviewed-on: http://review.gluster.org/8014
Reviewed-by: Rajesh Joseph <rjoseph@redhat.com>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Raghavendra Bhat <raghavendra@redhat.com>
Reviewed-by: Kaushal M <kaushal@redhat.com>
Diffstat (limited to 'xlators')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-snapshot.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-snapshot.c b/xlators/mgmt/glusterd/src/glusterd-snapshot.c index cca94f17133..7d1af47825b 100644 --- a/xlators/mgmt/glusterd/src/glusterd-snapshot.c +++ b/xlators/mgmt/glusterd/src/glusterd-snapshot.c @@ -7043,6 +7043,10 @@ glusterd_snapshot_postvalidate (dict_t *dict, int32_t op_ret, char **op_errstr, break; case GF_SNAP_OPTION_TYPE_ACTIVATE: case GF_SNAP_OPTION_TYPE_DEACTIVATE: + case GF_SNAP_OPTION_TYPE_STATUS: + case GF_SNAP_OPTION_TYPE_CONFIG: + case GF_SNAP_OPTION_TYPE_INFO: + case GF_SNAP_OPTION_TYPE_LIST: /*Nothing to be done. But want to * avoid the default case warning*/ ret = 0; |