diff options
| author | Vijaikumar M <vmallika@redhat.com> | 2014-05-19 13:09:07 +0530 |
|---|---|---|
| committer | Krishnan Parthasarathi <kparthas@redhat.com> | 2014-05-29 23:59:58 -0700 |
| commit | 7b96fd92de79f367bfa8591becdb30340a4a26e7 (patch) | |
| tree | adfd692f8a675a64a4ef4cbb29f77001cebdf1d8 /xlators/mgmt/glusterd/src/glusterd-handler.c | |
| parent | ac43be9d9c1925a5fddf21ba2061f414e53140a8 (diff) | |
cli: 'Snapshot Volume: yes/no' for volume info needs to be removed
With initial design where the snap volume used to be displayed in
gluster volume info,
we used "Snap Volume: yes/on" to distinguish the volume whether its a
snap volume or the original volume.
But with new design the snap volumes are not listed in the volume info,
hence this entry (snap volume:) doesn't make sense to show.
Change-Id: Ic5b9948bf4ef74e89a611742c74a8989cb406866
BUG: 1098910
Signed-off-by: Vijaikumar M <vmallika@redhat.com>
Reviewed-on: http://review.gluster.org/7794
Reviewed-by: Avra Sengupta <asengupt@redhat.com>
Reviewed-by: Sachin Pandit <spandit@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Rajesh Joseph <rjoseph@redhat.com>
Reviewed-by: Krishnan Parthasarathi <kparthas@redhat.com>
Tested-by: Krishnan Parthasarathi <kparthas@redhat.com>
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-handler.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-handler.c | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c index 64c7ddfa5fb..7303622aed7 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handler.c +++ b/xlators/mgmt/glusterd/src/glusterd-handler.c @@ -378,21 +378,6 @@ glusterd_add_volume_detail_to_dict (glusterd_volinfo_t *volinfo, if (ret) goto out; - /* As of now, the snap volumes are also displayed as part of - volume info command. So this change is to display whether - the volume is original volume or the snap_volume. If - displaying of snap volumes in volume info o/p is not needed - this should be removed. - */ - snprintf (key, 256, "volume%d.snap_volume", count); - ret = dict_set_int32 (volumes, key, volinfo->is_snap_volume); - if (ret) { - gf_log (this->name, GF_LOG_WARNING, "failed to set whether " - "the volume is a snap volume or actual volume (%s)", - volinfo->volname); - goto out; - } - snprintf (key, 256, "volume%d.brick_count", count); ret = dict_set_int32 (volumes, key, volinfo->brick_count); if (ret) |