summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatt Woodson <mwoodson@gmail.com>2015-09-22 16:00:37 -0400
committerMatt Woodson <mwoodson@gmail.com>2015-09-22 16:00:37 -0400
commitfd024841598adf3c77e57f13a17ff34a9955d43a (patch)
tree5f98cc99a782d32b3b6413200c5ce61e16b9d76b
parent8d2b270d3d8c53d0afc2285408da9e479c5974fc (diff)
parent984c3f7e47039be37b700536367d26ca02b61522 (diff)
downloadopenshift-fd024841598adf3c77e57f13a17ff34a9955d43a.tar.gz
openshift-fd024841598adf3c77e57f13a17ff34a9955d43a.tar.bz2
openshift-fd024841598adf3c77e57f13a17ff34a9955d43a.tar.xz
openshift-fd024841598adf3c77e57f13a17ff34a9955d43a.zip
Merge pull request #605 from kwoodson/tmp2
Adding support for prototypes and discoveryrules
-rw-r--r--roles/lib_zabbix/library/zbx_discoveryrule.py2
-rw-r--r--roles/lib_zabbix/library/zbx_itemprototype.py13
2 files changed, 11 insertions, 4 deletions
diff --git a/roles/lib_zabbix/library/zbx_discoveryrule.py b/roles/lib_zabbix/library/zbx_discoveryrule.py
index f32267818..f52f350a5 100644
--- a/roles/lib_zabbix/library/zbx_discoveryrule.py
+++ b/roles/lib_zabbix/library/zbx_discoveryrule.py
@@ -94,6 +94,7 @@ def main():
zbx_debug=dict(default=False, type='bool'),
name=dict(default=None, type='str'),
key=dict(default=None, type='str'),
+ description=dict(default=None, type='str'),
interfaceid=dict(default=None, type='int'),
ztype=dict(default='trapper', type='str'),
delay=dict(default=60, type='int'),
@@ -151,6 +152,7 @@ def main():
'interfaceid': module.params['interfaceid'],
'lifetime': module.params['lifetime'],
'type': get_type(module.params['ztype']),
+ 'description': module.params['description'],
}
if params['type'] in [2, 5, 7, 11]:
params.pop('interfaceid')
diff --git a/roles/lib_zabbix/library/zbx_itemprototype.py b/roles/lib_zabbix/library/zbx_itemprototype.py
index 75b9e25b7..4ec1b8e02 100644
--- a/roles/lib_zabbix/library/zbx_itemprototype.py
+++ b/roles/lib_zabbix/library/zbx_itemprototype.py
@@ -38,12 +38,12 @@ def exists(content, key='result'):
return True
-def get_rule_id(zapi, discoveryrule_name, templateid):
+def get_rule_id(zapi, discoveryrule_key, templateid):
'''get a discoveryrule by name
'''
content = zapi.get_content('discoveryrule',
'get',
- {'search': {'name': discoveryrule_name},
+ {'search': {'key_': discoveryrule_key},
'output': 'extend',
'templateids': templateid,
})
@@ -152,6 +152,7 @@ def main():
zbx_debug=dict(default=False, type='bool'),
name=dict(default=None, type='str'),
key=dict(default=None, type='str'),
+ description=dict(default=None, type='str'),
interfaceid=dict(default=None, type='int'),
ztype=dict(default='trapper', type='str'),
value_type=dict(default='float', type='str'),
@@ -160,6 +161,8 @@ def main():
state=dict(default='present', type='str'),
status=dict(default='enabled', type='str'),
applications=dict(default=[], type='list'),
+ template_name=dict(default=None, type='str'),
+ discoveryrule_key=dict(default=None, type='str'),
),
#supports_check_mode=True
)
@@ -173,6 +176,7 @@ def main():
zbx_class_name = 'itemprototype'
idname = "itemid"
state = module.params['state']
+ template = get_template(zapi, module.params['template_name'])
# selectInterfaces doesn't appear to be working but is needed.
content = zapi.get_content(zbx_class_name,
@@ -203,12 +207,13 @@ def main():
if state == 'present':
params = {'name': module.params['name'],
'key_': module.params['key'],
- 'hostid': content['result'][0]['hostid'],
+ 'hostid': template['templateid'],
'interfaceid': module.params['interfaceid'],
- 'ruleid': content['result'][0]['discoveryRule']['itemid'],
+ 'ruleid': get_rule_id(zapi, module.params['discoveryrule_key'], template['templateid']),
'type': get_type(module.params['ztype']),
'value_type': get_value_type(module.params['value_type']),
'applications': get_app_ids(zapi, module.params['applications']),
+ 'description': module.params['description'],
}
if params['type'] in [2, 5, 7, 8, 11, 15]: