geo-rep: add support to glob patterns with "geo-rep config"

Change-Id: I0d54cea72e4363eab85ade774cc918081d8036e9
BUG: 3610
Reviewed-on: http://review.gluster.com/489
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vijay@gluster.com>

2 files changed, 14 insertions, 6 deletions

diff --git a/xlators/features/marker/utils/syncdaemon/gsyncd.py b/xlators/features/marker/utils/syncdaemon/gsyncd.py
index f3b5988ad..67a873cb5 100644
--- a/xlators/features/marker/utils/syncdaemon/gsyncd.py
+++ b/xlators/features/marker/utils/syncdaemon/gsyncd.py
@@ -9,6 +9,7 @@ import signal
 import select
 import optparse
 import fcntl
+import fnmatch
 from optparse import OptionParser, SUPPRESS_HELP
 from logging import Logger
 from errno import EEXIST, ENOENT
@@ -177,7 +178,7 @@ def main_i():
         op.add_option('--config-' + a,      metavar='OPT',  type=str, dest='config', action='callback',
                       callback=store_local_obj(a, lambda vx: {'opt': vx}))
     op.add_option('--config-get-all', dest='config', action='callback', callback=store_local_obj('get', lambda vx: {'opt': None}))
-    for m in ('', '-rx'):
+    for m in ('', '-rx', '-glob'):
         # call this code 'Pythonic' eh?
         # have to define a one-shot local function to be able to inject (a value depending on the)
         # iteration variable into the inner lambda
@@ -186,7 +187,7 @@ def main_i():
                           callback=store_local_obj('set', lambda vx: {'opt': vx[0], 'val': vx[1], 'rx': rx}))
             op.add_option('--config-del' + m,   metavar='OPT',  type=str, dest='config', action='callback',
                           callback=store_local_obj('del', lambda vx: {'opt': vx, 'rx': rx}))
-        conf_mod_opt_regex_variant(not not m)
+        conf_mod_opt_regex_variant(m and m[1:] or False)
 
     op.add_option('--normalize-url',           dest='url_print', action='callback', callback=store_local_curry('normal'))
     op.add_option('--canonicalize-url',        dest='url_print', action='callback', callback=store_local_curry('canon'))
@@ -226,8 +227,11 @@ def main_i():
                 raise GsyncdError('tunable %s is not set to value %s required for restricted SSH invocaton' % \
                                   (k, v))
 
-    if getattr(confdata, 'rx', None):
+    confrx = getattr(confdata, 'rx', None)
+    if confrx:
         # peers are regexen, don't try to parse them
+        if confrx == 'glob':
+            args = [ '\A' + fnmatch.translate(a) for a in args ]
         canon_peers = args
         namedict = {}
     else:
diff --git a/xlators/mgmt/glusterd/src/glusterd-geo-rep.c b/xlators/mgmt/glusterd/src/glusterd-geo-rep.c
index 19975a689..eae5dd827 100644
--- a/xlators/mgmt/glusterd/src/glusterd-geo-rep.c
+++ b/xlators/mgmt/glusterd/src/glusterd-geo-rep.c
@@ -484,6 +484,7 @@ gsync_verify_config_options (dict_t *dict, char **op_errstr)
         char   *slave     = NULL;
         char   *op_name   = NULL;
         char   *op_value  = NULL;
+        char   *t         = NULL;
         gf_boolean_t banned = _gf_true;
 
         if (dict_get_str (dict, "subop", &subop) != 0) {
@@ -517,13 +518,16 @@ gsync_verify_config_options (dict_t *dict, char **op_errstr)
         if (strcmp (subop, "get") == 0)
                 return 0;
 
-        if (strcmp (subop, "set") != 0 && strcmp (subop, "del") != 0) {
+        t = strtail (subop, "set");
+        if (!t)
+                t = strtail (subop, "del");
+        if (!t || (t[0] && strcmp (t, "-glob") != 0)) {
                 gf_log ("", GF_LOG_WARNING, "unknown subop %s", subop);
                 *op_errstr = gf_strdup ("Invalid config request");
                 return -1;
         }
 
-        if (strcmp (subop, "set") == 0 &&
+        if (strtail (subop, "set") &&
             dict_get_str (dict, "op_value", &op_value) != 0) {
                 gf_log ("", GF_LOG_WARNING, "missing value for set");
                 *op_errstr = gf_strdup ("missing value");
@@ -1105,7 +1109,7 @@ glusterd_gsync_configure (glusterd_volinfo_t *volinfo, char *slave,
         if (ret != 0)
                 goto out;
 
-        if (strcmp (subop, "set") == 0) {
+        if (strtail (subop, "set")) {
                 ret = dict_get_str (dict, "op_value", &op_value);
                 if (ret != 0)
                         goto out;