From 2e56bde3ea952beabd27cdf8a3a10da563a00bcc Mon Sep 17 00:00:00 2001 From: Avra Sengupta Date: Wed, 28 Oct 2015 15:18:07 +0530 Subject: snapshot: Don't display snapshot's hard-limit and soft-limit in vol info The snap-max-hard-limit being displayed in the volume info currently is propagated from system's snap-max-hard-limit as that is a global option common for all volumes, and hence ends up showing the system's snap-max-hard-limit. We should not be displaying snap-max-hard-limit and snap-max-soft-limit in the volume info at all, as these are snap config options and should be set and displayed via snap config command. Modified bug-1113476.t to test the same behaviour. Change-Id: I90891f0cf7fb39fd686787297c7f7cd8c1e7daa1 BUG: 1276018 Signed-off-by: Avra Sengupta Reviewed-on: http://review.gluster.org/12443 Tested-by: Gluster Build System Reviewed-by: mohammed rafi kc Reviewed-by: Rajesh Joseph Tested-by: NetBSD Build System --- xlators/mgmt/glusterd/src/glusterd-handler.c | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'xlators/mgmt/glusterd/src') diff --git a/xlators/mgmt/glusterd/src/glusterd-handler.c b/xlators/mgmt/glusterd/src/glusterd-handler.c index 24e0084a093..d3f3be20a9b 100644 --- a/xlators/mgmt/glusterd/src/glusterd-handler.c +++ b/xlators/mgmt/glusterd/src/glusterd-handler.c @@ -307,6 +307,15 @@ _build_option_key (dict_t *d, char *k, data_t *v, void *tmp) (strcmp (k, "features.soft-limit") == 0)) return 0; } + + /* snap-max-hard-limit and snap-max-soft-limit are system * + * options set and managed by snapshot config option. Hence * + * they should not be displayed in gluster volume info. * + */ + if ((strcmp (k, "snap-max-hard-limit") == 0) || + (strcmp (k, "snap-max-soft-limit") == 0)) + return 0; + snprintf (reconfig_key, 256, "volume%d.option.%s", pack->vol_count, k); ret = dict_set_str (pack->dict, reconfig_key, v->data); -- cgit