commit b6a1a192955da3d18f50dfa3130c95fc15d7a66a
Author: Matthew Booth <email address hidden>
Date: Sat Dec 16 20:27:08 2017 +0000
Fix an error in _get_host_states when deleting a compute node
_get_host_states returns a generator which closes over seen_nodes, which
is local, and self.host_state_map, which is global. It also modifies
self.host_state_map, and will remove entries whose compute nodes are no
longer present.
If a compute node is deleted while a filter is still evaluating the
generator returned by _get_host_states, the entry in self.host_state_map
will be deleted if _get_host_states is called again. This will cause a
KeyError when the first generator comes to evaluate the entry for the
deleted compute node.
We fix this by modifying the returned generator expression to check
that a host_state_map entry still exists before returning it. An
existing unit test is modified to exhibit the bug.
Change-Id: Ibb7c43a0abc433f93fc3de71146263e6d5923666
Closes-Bug: #1739323
(cherry picked from commit d72b33b986525a9b2c7aa08b609ae386de1d0e89)
Reviewed: https:/ /review. openstack. org/529626 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=b6a1a192955 da3d18f50dfa313 0c95fc15d7a66a
Committed: https:/
Submitter: Zuul
Branch: stable/pike
commit b6a1a192955da3d 18f50dfa3130c95 fc15d7a66a
Author: Matthew Booth <email address hidden>
Date: Sat Dec 16 20:27:08 2017 +0000
Fix an error in _get_host_states when deleting a compute node
_get_ host_states returns a generator which closes over seen_nodes, which state_map, which is global. It also modifies host_state_ map, and will remove entries whose compute nodes are no
is local, and self.host_
self.
longer present.
If a compute node is deleted while a filter is still evaluating the
generator returned by _get_host_states, the entry in self.host_state_map
will be deleted if _get_host_states is called again. This will cause a
KeyError when the first generator comes to evaluate the entry for the
deleted compute node.
We fix this by modifying the returned generator expression to check
that a host_state_map entry still exists before returning it. An
existing unit test is modified to exhibit the bug.
Change-Id: Ibb7c43a0abc433 f93fc3de7114626 3e6d5923666 b2c7aa08b609ae3 86de1d0e89)
Closes-Bug: #1739323
(cherry picked from commit d72b33b986525a9