mgr/influx: handle ValueError when setting options

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/pybind/mgr/influx/module.py b/src/pybind/mgr/influx/module.py
index e54ca1f00244caa382c77272e2d63ed89fa68661..bcc82e1b9605181f452025e5b9e2511af9d86af7 100644 (file)
--- a/src/pybind/mgr/influx/module.py
+++ b/src/pybind/mgr/influx/module.py
@@ -448,9 +448,12 @@ class Module(MgrModule):
                 return -errno.EINVAL, '', 'Value should not be empty or None'
 
             self.log.debug('Setting configuration option %s to %s', key, value)
-            self.set_config_option(key, value)
-            self.set_module_option(key, value)
-            return 0, 'Configuration option {0} updated'.format(key), ''
+            try:
+                self.set_module_option(key, value)
+                self.set_config_option(key, value)
+                return 0, 'Configuration option {0} updated'.format(key), ''
+            except ValueError as e:
+                return -errno.EINVAL, '', str(e)
         elif cmd['prefix'] == 'influx send':
             self.send_to_influx()
             return 0, 'Sending data to Influx', ''