uselist should be True to DVRPortBinding orm.relationship

[ZongKai LI]

In DVR scenario, after a router interface has been bound to multiple hosts, when we remove this interface from router, in neutron server log, SQL warning will raise:
  SAWarning: Multiple rows returned with uselist=False for eagerly-loaded attribute 'Port.dvr_port_binding'

it's caused by https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/models.py#L130, uselist is set to False. But indeed, table ml2_dvr_port_bindings stores all bindings for router_interface_distributed ports, and for a that kind of port, it could have multiple bindings. So it's not a one-to-one relationship, we should remove "uselist=False" in DVRPortBinding port orm.relationship.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/274550

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/274550
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=951dd5e015cbd102aed80a6808f623b4626e727d
Submitter: Jenkins
Branch: master

commit 951dd5e015cbd102aed80a6808f623b4626e727d
Author: lzklibj <email address hidden>
Date: Mon Feb 1 16:39:54 2016 +0800

    Fix port relationship for DVRPortBinding

    uselist=False is used for one to one relationship, but for
    DVRPortBinding, a router_interface_distributed port can have
    multiple bindings. That causes SAWarning:
    Multiple rows returned with uselist=False for lazily-loaded attribute
    'Port.dvr_port_binding'.

    "uselist=False" is misused in DVRPortBinding port relationship,
    this patch will fix this.

    Change-Id: I2b00d96aaa445e0977bc3d4957d35a28d44dd953
    Closes-Bug: #1540259

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/liberty
Review: https://review.openstack.org/285385

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/neutron 8.0.0.0b3 development milestone.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/285385
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=0bce029749d931d19926504478aec1dfea9b52d5
Submitter: Jenkins
Branch: stable/liberty

commit 0bce029749d931d19926504478aec1dfea9b52d5
Author: lzklibj <email address hidden>
Date: Mon Feb 1 16:39:54 2016 +0800

    Fix port relationship for DVRPortBinding

    uselist=False is used for one to one relationship, but for
    DVRPortBinding, a router_interface_distributed port can have
    multiple bindings. That causes SAWarning:
    Multiple rows returned with uselist=False for lazily-loaded attribute
    'Port.dvr_port_binding'.

    "uselist=False" is misused in DVRPortBinding port relationship,
    this patch will fix this.

    Change-Id: I2b00d96aaa445e0977bc3d4957d35a28d44dd953
    Closes-Bug: #1540259
    (cherry picked from commit 951dd5e015cbd102aed80a6808f623b4626e727d)

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/neutron 7.1.0 release.