vmnames getter

PVE-Zabbix.py

@@ -1,11 +1,21 @@
 from PVEApi import PVEApi
+import json
 
 USERNAME = "zabbix@pve"
 PASSWORD = "zabbix"
 HOST = "172.16.3.11:8006"
+NODE='h1'
 
 conn = PVEApi(HOST, USERNAME, PASSWORD)
 
-print conn.get_node_list()
+
-for vm in conn.get_cluster_vmlist():
+def get_vmnames(vmlist):
-    print vm.node, vm.name, vm.vmid, vm.cpus
+    name_vmlist = [{"#VMNAME": vm.name} for vm in vmlist]
+    return name_vmlist
+
+def listofdicts_to_zabbix_json(list):
+    # TODO wrap in "data:[]"
+    zabbixjson=json.dumps(list)
+    return zabbixjson
+
+print listofdicts_to_zabbix_json(get_vmnames(conn.get_cluster_vmlist()))

PVEApi.py

@@ -43,7 +43,6 @@ class PVEApi(object):
             url = self.url + '/nodes/' + node + '/qemu'
             response = json.load(urllib2.urlopen(self.get_request(url), context=ssl._create_unverified_context()))
             resources = response['data']
-            print resources
             for machine in resources:
                 name = machine[u'name']
                 status = machine[u'status']

VM.py

@@ -20,4 +20,4 @@ class VM(object):
         self.maxmem = maxmem
 
     def __repr__(self):
-        return "<%s:%s>" % (self.node, self.name)
+        return "<%s-%s:%s>" % (self.node,self.vmid, self.name)