|
 |
ca3909 |
From 8b5b3b247a00515d3188453c27b0ba749e93d325 Mon Sep 17 00:00:00 2001
|
|
|
ca3909 |
From: Aravinda VK <avishwan@redhat.com>
|
|
|
ca3909 |
Date: Tue, 26 Mar 2019 13:20:13 +0530
|
|
|
ca3909 |
Subject: [PATCH 339/344] geo-rep: fix integer config validation
|
|
|
ca3909 |
|
|
|
ca3909 |
ssh-port validation is mentioned as `validation=int` in template
|
|
|
ca3909 |
`gsyncd.conf`, but not handled this during geo-rep config set.
|
|
|
ca3909 |
|
|
|
ca3909 |
upstream patch:
|
|
|
ca3909 |
https://review.gluster.org/#/c/glusterfs/+/22418/
|
|
|
ca3909 |
Backport of:
|
|
|
ca3909 |
|
|
|
ca3909 |
>Fixes: bz#1692666
|
|
|
ca3909 |
>Change-Id: I3f19d9b471b0a3327e4d094dfbefcc58ed2c34f6
|
|
|
ca3909 |
>Signed-off-by: Aravinda VK <avishwan@redhat.com>
|
|
|
ca3909 |
>Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
|
|
|
ca3909 |
|
|
|
ca3909 |
BUG: 1782162
|
|
|
ca3909 |
Change-Id: I3f19d9b471b0a3327e4d094dfbefcc58ed2c34f6
|
|
|
ca3909 |
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
|
|
|
ca3909 |
Reviewed-on: https://code.engineering.redhat.com/gerrit/187533
|
|
|
ca3909 |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
ca3909 |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
ca3909 |
---
|
|
|
ca3909 |
geo-replication/syncdaemon/gsyncdconfig.py | 23 ++++++++++++++++++-----
|
|
|
ca3909 |
tests/00-geo-rep/georep-basic-dr-rsync.t | 3 +++
|
|
|
ca3909 |
2 files changed, 21 insertions(+), 5 deletions(-)
|
|
|
ca3909 |
|
|
|
ca3909 |
diff --git a/geo-replication/syncdaemon/gsyncdconfig.py b/geo-replication/syncdaemon/gsyncdconfig.py
|
|
|
ca3909 |
index f823311..8848071 100644
|
|
|
ca3909 |
--- a/geo-replication/syncdaemon/gsyncdconfig.py
|
|
|
ca3909 |
+++ b/geo-replication/syncdaemon/gsyncdconfig.py
|
|
|
ca3909 |
@@ -329,6 +329,9 @@ class Gconf(object):
|
|
|
ca3909 |
if item["validation"] == "unixtime":
|
|
|
ca3909 |
return validate_unixtime(value)
|
|
|
ca3909 |
|
|
|
ca3909 |
+ if item["validation"] == "int":
|
|
|
ca3909 |
+ return validate_int(value)
|
|
|
ca3909 |
+
|
|
|
ca3909 |
return False
|
|
|
ca3909 |
|
|
|
ca3909 |
def _is_config_changed(self):
|
|
|
ca3909 |
@@ -381,6 +384,14 @@ def config_upgrade(config_file, ret):
|
|
|
ca3909 |
config.write(configfile)
|
|
|
ca3909 |
|
|
|
ca3909 |
|
|
|
ca3909 |
+def validate_int(value):
|
|
|
ca3909 |
+ try:
|
|
|
ca3909 |
+ _ = int(value)
|
|
|
ca3909 |
+ return True
|
|
|
ca3909 |
+ except ValueError:
|
|
|
ca3909 |
+ return False
|
|
|
ca3909 |
+
|
|
|
ca3909 |
+
|
|
|
ca3909 |
def validate_unixtime(value):
|
|
|
ca3909 |
try:
|
|
|
ca3909 |
y = datetime.fromtimestamp(int(value)).strftime("%Y")
|
|
|
ca3909 |
@@ -393,11 +404,13 @@ def validate_unixtime(value):
|
|
|
ca3909 |
|
|
|
ca3909 |
|
|
|
ca3909 |
def validate_minmax(value, minval, maxval):
|
|
|
ca3909 |
- value = int(value)
|
|
|
ca3909 |
- minval = int(minval)
|
|
|
ca3909 |
- maxval = int(maxval)
|
|
|
ca3909 |
-
|
|
|
ca3909 |
- return value >= minval and value <= maxval
|
|
|
ca3909 |
+ try:
|
|
|
ca3909 |
+ value = int(value)
|
|
|
ca3909 |
+ minval = int(minval)
|
|
|
ca3909 |
+ maxval = int(maxval)
|
|
|
ca3909 |
+ return value >= minval and value <= maxval
|
|
|
ca3909 |
+ except ValueError:
|
|
|
ca3909 |
+ return False
|
|
|
ca3909 |
|
|
|
ca3909 |
|
|
|
ca3909 |
def validate_choice(value, allowed_values):
|
|
|
ca3909 |
diff --git a/tests/00-geo-rep/georep-basic-dr-rsync.t b/tests/00-geo-rep/georep-basic-dr-rsync.t
|
|
|
ca3909 |
index b432635..b6fbf18 100644
|
|
|
ca3909 |
--- a/tests/00-geo-rep/georep-basic-dr-rsync.t
|
|
|
ca3909 |
+++ b/tests/00-geo-rep/georep-basic-dr-rsync.t
|
|
|
ca3909 |
@@ -71,6 +71,9 @@ EXPECT_WITHIN $GEO_REP_TIMEOUT 4 check_status_num_rows "Created"
|
|
|
ca3909 |
#Config gluster-command-dir
|
|
|
ca3909 |
TEST $GEOREP_CLI $master $slave config gluster-command-dir ${GLUSTER_CMD_DIR}
|
|
|
ca3909 |
|
|
|
ca3909 |
+#Config Set ssh-port to validate int validation
|
|
|
ca3909 |
+TEST $GEOREP_CLI $master $slave config ssh-port 22
|
|
|
ca3909 |
+
|
|
|
ca3909 |
#Config gluster-command-dir
|
|
|
ca3909 |
TEST $GEOREP_CLI $master $slave config slave-gluster-command-dir ${GLUSTER_CMD_DIR}
|
|
|
ca3909 |
|
|
|
ca3909 |
--
|
|
|
ca3909 |
1.8.3.1
|
|
|
ca3909 |
|