Comment 55 for bug 1253896

Reviewed: https://review.openstack.org/65838
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=2702baed390d094b0eac07d0ae167ed236868d00
Submitter: Jenkins
Branch: master

commit 2702baed390d094b0eac07d0ae167ed236868d00
Author: Salvatore Orlando <email address hidden>
Date: Mon Jan 13 12:51:03 2014 -0800

    Avoid processing ports which are not yet ready

    This patch changes get_vif_port_set in order to not return
    OVS ports for which the ofport is not yet assigned, thus avoiding
    a regex match failure in get_vif_port_by_id.
    Because of this failure, treat_vif_port is unable to wire
    the port.
    As get_vif_port_by_id is also used elsewhere in the agent, it has
    been enhanced in order to tolerate situations in which ofport might
    have not yet been assigned.

    The ofport field is added to the list of those monitored by the
    SimpleInterfaceMonitor. This will guarantee an event is generated
    when the ofport is assigned to a port. Otherwise there is a risk
    a port would be never processed if it was not yet ready the first
    time is was detected. This change won't trigger any extra processing
    on the agent side.

    Finally, this patch avoids fetching device details from the plugin
    for ports which have disappeared from the OVS bridge. This is a
    little optimization which might be beneficial for short lived ports.

    Change-Id: Icf7f0c7d6fe5239a358567cc9dc9db8ec11c15be
    Partial-Bug: #1253896