get_vm_qp in vm.py throws an exception if the instance no longer exists

[Kris Kendall]

(Pdb) w
  /usr/local/lib/python2.7/dist-packages/eventlet/green/thread.py(41)__thread_body()
-> func(*args, **kwargs)
  /usr/lib/python2.7/threading.py(774)__bootstrap()
-> self.__bootstrap_inner()
  /usr/lib/python2.7/threading.py(801)__bootstrap_inner()
-> self.run()
  /usr/local/lib/python2.7/dist-packages/pypowervm/adapter.py(1685)run()
-> self.eventlistener._dispatch_events(events, raw_events)
  /usr/local/lib/python2.7/dist-packages/pypowervm/adapter.py(1591)_dispatch_events()
-> call_handlers()
  /usr/local/lib/python2.7/dist-packages/pypowervm/adapter.py(1582)call_handlers()
-> h.process(raw_events)
  /opt/stack/nova-powervm/nova_powervm/virt/powervm/driver.py(1994)process()
-> None))
  /opt/stack/nova-powervm/nova_powervm/virt/powervm/driver.py(1901)_handle_event()
-> inst, pvm_uuid, uri, etype, details, eid)
  /opt/stack/nova-powervm/nova_powervm/virt/powervm/driver.py(1924)_handle_inst_event()
-> 'PartitionState')
> /opt/stack/nova-powervm/nova_powervm/virt/powervm/vm.py(503)get_vm_qp()

[Eric Fried (efried)]

Any update on this, Ravi? Did you propose a patch at some point? Is this still an issue since Iaadca830b11318787868a42e00fc814241d18081 ?

[Matthew Edmonds (edmondsw)]

This appears to be fixed. The code from the original trace [1] has been refactored into a different file [2] and subsequently modified to no longer call get_vm_qp [3].

[1] https://github.com/openstack/nova-powervm/blob/f560b5913359ea3dbc789d63069d8e1bbbef7a10/nova_powervm/virt/powervm/driver.py#L1922
[2] https://github.com/openstack/nova-powervm/commit/3a78c42ff0040df5a60531d536e8f1689659a696#diff-3841d4d745c5150c8ece02ab5d1159a4R86
[3] https://github.com/openstack/nova-powervm/commit/db759ce5158446a918e76c049d0efc753e0bbc72#diff-3841d4d745c5150c8ece02ab5d1159a4R107