DHCP agent notifying all ports ready on sync state

[Kevin Benton]

The DHCP agent is notifying the server every port is ready that it serves after running the sync_state command, even if it's only for one network. This is an expensive waste of resources because the DHCP agent might resync one network and then have to notify the server of thousands of ports.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/379270
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=7dc2707c2b0b05ad7699a24680a5b45e67f717fc
Submitter: Jenkins
Branch: master

commit 7dc2707c2b0b05ad7699a24680a5b45e67f717fc
Author: Kevin Benton <email address hidden>
Date: Thu Sep 29 01:37:59 2016 -0700

    Only mark ports ready on synced networks

    The DHCP agent was previously resending every single port to
    the server whenever sync_state was called, even if it was just
    for one network.

    This let to sending way too much unnecessary data to the server
    and also potentially resulted in sending a port to the server
    that wasn't actually provisioned yet.

    This patch corrects the behavior by only sending ports for networks
    that are being synced if it's a conditional sync.

    Closes-Bug: #1639086
    Change-Id: Ie7686837b18ff251baa315ef95dc511cda475672

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/394072

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/394072
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=909ed04c96fdf46d39645a79c7ef4c988fa04877
Submitter: Jenkins
Branch: stable/newton

commit 909ed04c96fdf46d39645a79c7ef4c988fa04877
Author: Kevin Benton <email address hidden>
Date: Thu Sep 29 01:37:59 2016 -0700

    Only mark ports ready on synced networks

    The DHCP agent was previously resending every single port to
    the server whenever sync_state was called, even if it was just
    for one network.

    This let to sending way too much unnecessary data to the server
    and also potentially resulted in sending a port to the server
    that wasn't actually provisioned yet.

    This patch corrects the behavior by only sending ports for networks
    that are being synced if it's a conditional sync.

    Closes-Bug: #1639086
    Change-Id: Ie7686837b18ff251baa315ef95dc511cda475672
    (cherry picked from commit 7dc2707c2b0b05ad7699a24680a5b45e67f717fc)

[OpenStack Infra (hudson-openstack)]

This issue was fixed in the openstack/neutron 10.0.0.0b1 development milestone.

[OpenStack Infra (hudson-openstack)]

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