From f93761c547c39fad20f9aa96d4baf2641500b9a0 Mon Sep 17 00:00:00 2001 From: Atin Mukherjee Date: Tue, 26 Apr 2016 12:46:18 +0530 Subject: glusterd: fix validation of lower op-version check in volume set Commit 2d87a98 introduced a validation to fail lowering down the cluster.op-version. Commit 2eb8758 actually changed the variable value from cluster's op-version to volume's op-version which resulted the logic go for a toss. Change-Id: I70df32b75c3a3fe47dc840c4a655059e5b124bca BUG: 1315186 Signed-off-by: Atin Mukherjee Reviewed-on: http://review.gluster.org/14069 Smoke: Gluster Build System NetBSD-regression: NetBSD Build System CentOS-regression: Gluster Build System Reviewed-by: Kaushal M --- .../bug-1315186-reject-lowering-down-op-version.t | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) create mode 100644 tests/bugs/glusterd/bug-1315186-reject-lowering-down-op-version.t (limited to 'tests') diff --git a/tests/bugs/glusterd/bug-1315186-reject-lowering-down-op-version.t b/tests/bugs/glusterd/bug-1315186-reject-lowering-down-op-version.t new file mode 100644 index 00000000000..4bd6eaac59f --- /dev/null +++ b/tests/bugs/glusterd/bug-1315186-reject-lowering-down-op-version.t @@ -0,0 +1,22 @@ +#! /bin/bash + +. $(dirname $0)/../../include.rc + +# The test validates that lowering down the op-version should fail + +cleanup + +TEST glusterd +TEST pidof glusterd + +#volume create is just to ensure glusterd.info file is created +TEST $CLI volume create $V0 $H0:$B0/b1 + +GDWD=$($CLI system getwd) +OP_VERS_ORIG=$(grep 'operating-version' ${GDWD}/glusterd.info | cut -d '=' -f 2) +OP_VERS_NEW=`expr $OP_VERS_ORIG-1` + +TEST ! $CLI volume set all $V0 cluster.op-version $OP_VERS_NEW + +cleanup; + -- cgit