xenserver: Improve efficiency of code by using get_all_records_where()

Replace the get_record() for network references which caused as many slave-to-master calls as there are Network records plus one. The get_all_records_where() call gets exactly what is needed with a single call. Signed-off-by: Rob Hoes <[email protected]> Acked-by: Dominic Curran <[email protected]> Signed-off-by: Ben Pfaff <[email protected]>
vefimova · Jun 27, 2012 · 1dc6839 · 1dc6839
1 parent 0d7e2fe
commit 1dc6839
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync b/xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync
@@ -82,10 +82,9 @@ def get_network_by_bridge(br_name):
                 " XAPI session could not be initialized" % br_name)
         return None
 
-    for n in session.xenapi.network.get_all():
-        rec = session.xenapi.network.get_record(n)
-        if rec['bridge'] == br_name:
-            return rec
+    recs = session.xenapi.network.get_all_records_where('field "bridge"="%s"' % br_name)
+    if len(recs) > 0:
+        return recs.values()[0]
 
     return None