diff options
| author | Atin Mukherjee <amukherj@redhat.com> | 2019-04-09 14:58:29 +0530 |
|---|---|---|
| committer | Atin Mukherjee <amukherj@redhat.com> | 2019-04-09 15:16:31 +0530 |
| commit | efbf8abcc3bc729a90d4a7b57dc515f1df8a5863 (patch) | |
| tree | 41485de578e4a0c46cfe3efcafe111bde61afde2 | |
| parent | ae097b8ad5c160aed43d82187b623f053adf11b4 (diff) | |
glusterd: load ctime in the client graph only if it's not turned off
Considering ctime is a client side feature, we can't blindly load ctime
xlator into the client graph if it's explicitly turned off, that'd
result into backward compatibility issue where an old client can't mount
a volume configured on a server which is having ctime feature.
Fixes: bz#1697907
Change-Id: I6ae7b96d056073aa6746de9a449cf319786d45cc
Signed-off-by: Atin Mukherjee <amukherj@redhat.com>
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-volgen.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-volgen.c b/xlators/mgmt/glusterd/src/glusterd-volgen.c index 08b80fcf142..5e3af387bae 100644 --- a/xlators/mgmt/glusterd/src/glusterd-volgen.c +++ b/xlators/mgmt/glusterd/src/glusterd-volgen.c @@ -4301,7 +4301,8 @@ client_graph_builder(volgen_graph_t *graph, glusterd_volinfo_t *volinfo, } } - if (conf->op_version >= GD_OP_VERSION_5_0) { + if (conf->op_version >= GD_OP_VERSION_5_0 && + !dict_get_str_boolean(set_dict, "features.ctime", _gf_false)) { xl = volgen_graph_add(graph, "features/utime", volname); if (!xl) { ret = -1; |