|
 |
50dc83 |
From f1f27e5839dd99389bef65f79ea491e98e6935d2 Mon Sep 17 00:00:00 2001
|
|
|
50dc83 |
From: Ravishankar N <ravishankar@redhat.com>
|
|
|
50dc83 |
Date: Tue, 23 Apr 2019 18:05:36 +0530
|
|
|
50dc83 |
Subject: [PATCH 133/141] glusterd: enable fips-mode-rchecksum for new volumes
|
|
|
50dc83 |
|
|
|
50dc83 |
...during volume create if the cluster op-version is >=GD_OP_VERSION_7_0.
|
|
|
50dc83 |
|
|
|
50dc83 |
This option itself was introduced in GD_OP_VERSION_4_0_0 via commit 6daa65356.
|
|
|
50dc83 |
We missed enabling it by default for new volume creates in that commit.
|
|
|
50dc83 |
If we are to do it now safely, we need to use op version
|
|
|
50dc83 |
GD_OP_VERSION_7_0 and target it for release-7.
|
|
|
50dc83 |
|
|
|
50dc83 |
Patch in upstream master: https://review.gluster.org/#/c/glusterfs/+/22609/
|
|
|
50dc83 |
BUG: 1706683
|
|
|
50dc83 |
Change-Id: I7c6d4a8abe0816367e7069cb5cad01744f04858f
|
|
|
50dc83 |
fixes: bz#1706683
|
|
|
50dc83 |
Signed-off-by: Ravishankar N <ravishankar@redhat.com>
|
|
|
50dc83 |
Reviewed-on: https://code.engineering.redhat.com/gerrit/169443
|
|
|
50dc83 |
Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
|
|
|
50dc83 |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
50dc83 |
---
|
|
|
50dc83 |
xlators/mgmt/glusterd/src/glusterd-volgen.c | 13 +++++++++++++
|
|
|
50dc83 |
1 file changed, 13 insertions(+)
|
|
|
50dc83 |
|
|
|
50dc83 |
diff --git a/xlators/mgmt/glusterd/src/glusterd-volgen.c b/xlators/mgmt/glusterd/src/glusterd-volgen.c
|
|
|
50dc83 |
index da877aa..77aa705 100644
|
|
|
50dc83 |
--- a/xlators/mgmt/glusterd/src/glusterd-volgen.c
|
|
|
50dc83 |
+++ b/xlators/mgmt/glusterd/src/glusterd-volgen.c
|
|
|
50dc83 |
@@ -1614,10 +1614,17 @@ brick_graph_add_posix(volgen_graph_t *graph, glusterd_volinfo_t *volinfo,
|
|
|
50dc83 |
gf_boolean_t pgfid_feat = _gf_false;
|
|
|
50dc83 |
char *value = NULL;
|
|
|
50dc83 |
xlator_t *xl = NULL;
|
|
|
50dc83 |
+ xlator_t *this = NULL;
|
|
|
50dc83 |
+ glusterd_conf_t *priv = NULL;
|
|
|
50dc83 |
|
|
|
50dc83 |
if (!graph || !volinfo || !set_dict || !brickinfo)
|
|
|
50dc83 |
goto out;
|
|
|
50dc83 |
|
|
|
50dc83 |
+ this = THIS;
|
|
|
50dc83 |
+ GF_VALIDATE_OR_GOTO("glusterd", this, out);
|
|
|
50dc83 |
+ priv = this->private;
|
|
|
50dc83 |
+ GF_VALIDATE_OR_GOTO("glusterd", priv, out);
|
|
|
50dc83 |
+
|
|
|
50dc83 |
ret = glusterd_volinfo_get(volinfo, VKEY_FEATURES_QUOTA, &value);
|
|
|
50dc83 |
if (value) {
|
|
|
50dc83 |
ret = gf_string2boolean(value, "a_enabled);
|
|
|
50dc83 |
@@ -1661,6 +1668,12 @@ brick_graph_add_posix(volgen_graph_t *graph, glusterd_volinfo_t *volinfo,
|
|
|
50dc83 |
}
|
|
|
50dc83 |
}
|
|
|
50dc83 |
|
|
|
50dc83 |
+ if (priv->op_version >= GD_OP_VERSION_7_0) {
|
|
|
50dc83 |
+ ret = xlator_set_fixed_option(xl, "fips-mode-rchecksum", "on");
|
|
|
50dc83 |
+ if (ret) {
|
|
|
50dc83 |
+ goto out;
|
|
|
50dc83 |
+ }
|
|
|
50dc83 |
+ }
|
|
|
50dc83 |
snprintf(tmpstr, sizeof(tmpstr), "%d", brickinfo->fs_share_count);
|
|
|
50dc83 |
ret = xlator_set_fixed_option(xl, "shared-brick-count", tmpstr);
|
|
|
50dc83 |
out:
|
|
|
50dc83 |
--
|
|
|
50dc83 |
1.8.3.1
|
|
|
50dc83 |
|