From 7b96fd92de79f367bfa8591becdb30340a4a26e7 Mon Sep 17 00:00:00 2001 From: Vijaikumar M Date: Mon, 19 May 2014 13:09:07 +0530 Subject: 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 Reviewed-on: http://review.gluster.org/7794 Reviewed-by: Avra Sengupta Reviewed-by: Sachin Pandit Tested-by: Gluster Build System Reviewed-by: Rajesh Joseph Reviewed-by: Krishnan Parthasarathi Tested-by: Krishnan Parthasarathi --- xlators/mgmt/glusterd/src/glusterd-handler.c | 15 --------------- 1 file changed, 15 deletions(-) (limited to 'xlators') 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) -- cgit