Comment 19 for bug 1737892

Reviewed: https://review.openstack.org/635650
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=20fa3f267ebe7e2feb60d930af4a58a77cb9334c
Submitter: Zuul
Branch: stable/ocata

commit 20fa3f267ebe7e2feb60d930af4a58a77cb9334c
Author: Sławek Kapłoński <email address hidden>
Date: Fri Jan 12 23:08:56 2018 +0100

    [OVS] Fix for cleaning after skipped_devices

    When Openvswitch agent will get "port_update" event
    (e.g. to set port as unbound) and port is already removed
    from br-int when agent tries to get vif port in
    treat_devices_added_updated() method (port is removed
    because e.g. nova-compute removes it) then resources set
    for port by L2 agent extension drivers (like qos) are not
    cleaned properly.

    In such case port is added to skipped_ports and is set
    as DOWN in neutron-db but ext_manager is not called then
    for such port so it will not clear stuff like bandwidth
    limit's QoS and queue records and also DSCP marking
    open flow rules for this port.

    This patch fixes this issue by adding call of
    ext_manager.delete_port() method for all skipped ports.

    Change-Id: I3cf5c57c7f232deaa190ab6b0129e398fdabe592
    Closes-Bug: #1737892
    (cherry picked from commit a8271e978a1c540ae9888f568cf14b4c40ea1b6d)
    (cherry picked from commit cdcc704b9e0c6eb3eb8500e5791b007e85653c13)