commit 188f5c29a918e40379671d7d5dda729d3952ab64
Author: Jakub Libosvar <email address hidden>
Date: Thu Aug 31 12:54:47 2017 +0200
dvr: Don't raise KeyError in _get_floatingips_bound_to_host
We thought _get_floatingips_bound_to_host is not needed but removing the
method caused sending garps for fip that doesn't belong to node during
the full-sync.
This patch just replaces dict lookup with get() method, so fips are
filtered based on presence on the host and if host is not set on fip, it
won't raise a KeyError.
Note: This patch hasn't been merged in master yet because of KeyError happening in grenade job now.
Reviewed: https:/ /review. openstack. org/500077 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=188f5c29a91 8e40379671d7d5d da729d3952ab64
Committed: https:/
Submitter: Jenkins
Branch: stable/pike
commit 188f5c29a918e40 379671d7d5dda72 9d3952ab64
Author: Jakub Libosvar <email address hidden>
Date: Thu Aug 31 12:54:47 2017 +0200
dvr: Don't raise KeyError in _get_floatingip s_bound_ to_host
We thought _get_floatingip s_bound_ to_host is not needed but removing the
method caused sending garps for fip that doesn't belong to node during
the full-sync.
This patch just replaces dict lookup with get() method, so fips are
filtered based on presence on the host and if host is not set on fip, it
won't raise a KeyError.
Note: This patch hasn't been merged in master yet because of KeyError happening in grenade job now.
Co-Authored-By: Swaminathan Vasudevan <email address hidden>
Related-bug: #1712412
Related-bug: #1713927
Change-Id: I0fbc772d757fb1 3b788f9df8d6d7d 28d288d054a