diff options
| author | Ryan Ding <ryan.ding@open-fs.com> | 2016-11-16 10:52:34 +0800 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2017-08-16 17:01:32 +0000 |
| commit | 9b11b90d30285942113bb1e2d2a662337a75511c (patch) | |
| tree | 1aba99a6aff14c1b3f3ceb6231ee631acd4552fc /xlators/mgmt | |
| parent | def09c4fbb2805618715c5a125ddf482d7e53de3 (diff) | |
glusterd: Fix the debug and error-gen turned on issue
Problem:
When enabling one of the debug or the error-gen xlator, both of
them will be turned on together.
The problem is that when adding debug xlators into the graph, its
on/off status is ignored to be checked.
Fix:
When adding a debug xlator, check its status and skip those with
a value of "off".
Change-Id: Ifa79314ff337b60ee941cfb0308c1ccbe42d027f
BUG: 1395492
Signed-off-by: Zhang Huan <zhanghuan@open-fs.com>
Reviewed-on: https://review.gluster.org/15849
Tested-by: Atin Mukherjee <amukherj@redhat.com>
Smoke: Gluster Build System <jenkins@build.gluster.org>
CentOS-regression: Gluster Build System <jenkins@build.gluster.org>
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Diffstat (limited to 'xlators/mgmt')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-volgen.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-volgen.c b/xlators/mgmt/glusterd/src/glusterd-volgen.c index d82e0f1ae86..410885e82de 100644 --- a/xlators/mgmt/glusterd/src/glusterd-volgen.c +++ b/xlators/mgmt/glusterd/src/glusterd-volgen.c @@ -2554,6 +2554,8 @@ debugxl_option_handler (volgen_graph_t *graph, struct volopt_map_entry *vme, return 0; add_graph: + if (strcmp (vme->value, "off") == 0) + return 0; if (volgen_graph_add (graph, vme->voltype, volname)) return 0; else |