security_group.py: fix NameError: name 'd' is not defined (#3641)

in agent.log on a host running with cloudstack 4.14.0.0-SNAPSHOT, it gives error

root@node212:~# /usr/share/cloudstack-common/scripts/vm/network/security_group.py get_rule_logs_for_vms
Traceback (most recent call last):
  File "/usr/share/cloudstack-common/scripts/vm/network/security_group.py", line 1272, in <module>
    get_rule_logs_for_vms()
  File "/usr/share/cloudstack-common/scripts/vm/network/security_group.py", line 758, in get_rule_logs_for_vms
    vms = virshlist(state)
  File "/usr/share/cloudstack-common/scripts/vm/network/security_group.py", line 104, in virshlist
    alldomains = [d for domain in map(conn.lookupByID, conn.listDomainsID())]
  File "/usr/share/cloudstack-common/scripts/vm/network/security_group.py", line 104, in <listcomp>
    alldomains = [d for domain in map(conn.lookupByID, conn.listDomainsID())]
NameError: name 'd' is not defined
This commit is contained in:
Wei Zhou 2019-10-22 13:17:30 +02:00 committed by Rohit Yadav
parent 9258c7c3c9
commit 848860d822

View File

@ -101,8 +101,8 @@ def virshlist(states):
conn = get_libvirt_connection() conn = get_libvirt_connection()
alldomains = [d for domain in map(conn.lookupByID, conn.listDomainsID())] alldomains = [domain for domain in map(conn.lookupByID, conn.listDomainsID())]
alldomains += [d for domain in map(conn.lookupByName, conn.listDefinedDomains())] alldomains += [domain for domain in map(conn.lookupByName, conn.listDefinedDomains())]
domains = [] domains = []
for domain in alldomains: for domain in alldomains: