Comment 11 for bug 1712412

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/pike)

Reviewed: https://review.openstack.org/500077
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=188f5c29a918e40379671d7d5dda729d3952ab64
Submitter: Jenkins
Branch: stable/pike

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.

    Co-Authored-By: Swaminathan Vasudevan <email address hidden>

    Related-bug: #1712412
    Related-bug: #1713927

    Change-Id: I0fbc772d757fb13b788f9df8d6d7d28d288d054a