mir/openvswitch
2
0
Fork 0
mirror of https://github.com/openvswitch/ovs synced 2025-10-29 15:28:56 +00:00
Code Issues Releases Wiki Activity

xenserver: Reduce number of xapi DB calls in plugin

Browse Source 
When there are lots of PIFs in a XenServer/XCP pool, for example when
there are many VLANs configured on the pool, operations such as
PIF.get_all and loops over all PIFs which include database operations,
are very inefficient when executed on a pool slave, and should be
avoided as much as possible. This patch reduces the number of database
calls in the update function of the openvswitch-cfg-update xapi plugin.

Signed-off-by: Rob Hoes <rob.hoes@citrix.com>
Acked-by: Dominic Curran <Dominic.curran@citrix.com>
This commit is contained in:
Rob Hoes
2011-12-05 14:43:12 +00:00
committed by Ben Pfaff
parent f850000440
commit 3249bb907a
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
xenserver/etc_xapi.d_plugins_openvswitch-cfg-update
View File

@@ -86,8 +86,7 @@ def update(session, args):
bton = {}
for n in session.xenapi.network.get_all():
rec = session.xenapi.network.get_record(n)
for rec in session.xenapi.network.get_all_records().values():
try:
bton[rec['bridge']] = rec
except KeyError:
@@ -100,10 +99,9 @@ def update(session, args):
host_mgmt_device = None
pool_mgmt_macs = {}
if new_controller:
for n in session.xenapi.PIF.get_all():
rec = session.xenapi.PIF.get_record(n)
if rec.get('management', False):
pool_mgmt_macs[rec.get('MAC')] = rec.get('device')
recs = session.xenapi.PIF.get_all_records_where('field "management"="true"')
for rec in recs:
pool_mgmt_macs[rec.get('MAC')] = rec.get('device')
dib_changed = False
fail_mode_changed = False