Comment 48 for bug 1514056

Reviewed: https://review.openstack.org/300424
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=8d29f38356fc5b840fa8b5c31fcd9d76c0fdd336
Submitter: Jenkins
Branch: stable/liberty

commit 8d29f38356fc5b840fa8b5c31fcd9d76c0fdd336
Author: Hynek Mlnarik <email address hidden>
Date: Thu Feb 25 11:34:15 2016 +0100

    Cleanup stale OVS flows for physical bridges

    Perform deletion of the stale flows in physical bridges consistently with
    br-int and br-tun, respecting drop_flows_on_start configuration option.
    Added tests for auxiliary bridge and functional tests for the physical
    bridge using VLAN/flat external network. Fixes part of the bug 1514056;
    together with [1] and [2], the bug should be considered fixed.

    The commit also fixes inconsistency between netmask of allocated IP
    addresses assigned in _create_test_port_dict and ip_len in _plug_ports
    of base.py.

    Further, this commit sets agent UUID to physical bridges similarly to
    tun and int bridges. This is necessary for stale flows cleanup to work
    correctly. In upstream, it is treated using OVSBridgeCookieMixin.

    [1] https://review.openstack.org/#/c/297211/
    [2] https://review.openstack.org/#/c/297818/

    Conflicts:
     neutron/tests/functional/agent/l2/base.py
     neutron/tests/unit/plugins/ml2/drivers/openvswitch/agent/test_ovs_tunnel.py

    Co-Authored-By: Jian Wen <email address hidden>
    Co-Authored-By: Clayton O'Neill <email address hidden>
    Partial-Bug: 1514056
    Change-Id: I9801b76829021c9a0e6358982e1136637634a521
    (cherry picked from commit cacde308eef6f1d7005e555b4521332da95d3cf4)