Fix bug in commit
3249bb907a1dab9b0, which incorrectly assumed that
get_all_records_where() returned a list. It in fact returns a
dictionary and the list iteratory needs to change to account for this.
Thanks to Nicira for pointing this out.
NIC-454.
Reported-by: David Tsai <dtsai@nicira.com>
Acked-by: Rob Hoes <rob.hoes@citrix.com>
Signed-off-by: Dominic Curran <dominic.curran@citrix.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Daniel Roman droman@nicira.com
David Erickson derickso@stanford.edu
Devendra Naga devendra.aaru@gmail.com
Daniel Roman droman@nicira.com
David Erickson derickso@stanford.edu
Devendra Naga devendra.aaru@gmail.com
+Dominic Curran dominic.curran@citrix.com
Edward Tomasz Napierała trasz@freebsd.org
Ethan Jackson ethan@nicira.com
Gaetano Catalli gaetano.catalli@gmail.com
Edward Tomasz Napierała trasz@freebsd.org
Ethan Jackson ethan@nicira.com
Gaetano Catalli gaetano.catalli@gmail.com
Cedric Hobbs cedric@nicira.com
Dave Walker DaveWalker@ubuntu.com
Derek Cormier derek.cormier@lab.ntt.co.jp
Cedric Hobbs cedric@nicira.com
Dave Walker DaveWalker@ubuntu.com
Derek Cormier derek.cormier@lab.ntt.co.jp
-Dominic Curran dominic.curran@citrix.com
Duffie Cooley dcooley@nicira.com
DK Moon dkmoon@nicira.com
Edwin Chiu echiu@nicira.com
Duffie Cooley dcooley@nicira.com
DK Moon dkmoon@nicira.com
Edwin Chiu echiu@nicira.com
pool_mgmt_macs = {}
if new_controller:
recs = session.xenapi.PIF.get_all_records_where('field "management"="true"')
pool_mgmt_macs = {}
if new_controller:
recs = session.xenapi.PIF.get_all_records_where('field "management"="true"')
+ for rec in recs.itervalues():
pool_mgmt_macs[rec.get('MAC')] = rec.get('device')
dib_changed = False
pool_mgmt_macs[rec.get('MAC')] = rec.get('device')
dib_changed = False