mgr/zabbix: use Option for adding options

more type safety

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

diff --git a/src/pybind/mgr/zabbix/module.py b/src/pybind/mgr/zabbix/module.py
index d427038fa3107d2cbedf98c28798f4203196f696..74ebf7a277ff2b17eaa4286486abe7088d376aba 100644 (file)
--- a/src/pybind/mgr/zabbix/module.py
+++ b/src/pybind/mgr/zabbix/module.py
@@ -9,7 +9,7 @@ import errno
 import re
 from subprocess import Popen, PIPE
 from threading import Event
-from mgr_module import MgrModule
+from mgr_module import MgrModule, Option
 
 
 def avg(data):
@@ -60,33 +60,28 @@ class Module(MgrModule):
                 for o in self.MODULE_OPTIONS)
 
     MODULE_OPTIONS = [
-            {
-                'name': 'zabbix_sender',
-                'default': '/usr/bin/zabbix_sender'
-            },
-            {
-                'name': 'zabbix_host',
-                'default': None
-            },
-            {
-                'name': 'zabbix_port',
-                'type': 'int',
-                'default': 10051
-            },
-            {
-                'name': 'identifier',
-                'default': ""
-            },
-            {
-                'name': 'interval',
-                'type': 'secs',
-                'default': 60
-            },
-            {
-                'name': 'discovery_interval',
-                'type': 'count',
-                'default': 100
-            }
+        Option(
+            name='zabbix_sender',
+            default='/usr/bin/zabbix_sender'),
+        Option(
+            name='zabbix_host',
+            type='str',
+            default=None),
+        Option(
+            name='zabbix_port',
+            type='int',
+            default=10051),
+        Option(
+            name='identifier',
+            default=""),
+        Option(
+            name='interval',
+            type='secs',
+            default=60),
+        Option(
+            name='discovery_interval',
+            type='count',
+            default=100)
     ]
 
     COMMANDS = [