Neutron doesn't send notifications for network DHCP ports

[Steve McLellan]

Neutron has some magical automatic ports to allow various behaviors like attaching routers to subnets and allowing DHCP for subnets. There are no separate port notifications at this time, which means we can either a) ignore them (which seems bad) or b) listen for subnet notifications and look for an update the magic DHCP port. There's a similar situation with router interface ports. The device_owner values are 'network:dhcp' and 'network:router_interface' respectively. Router interfaces are handled in the subnet and router BP implementation (https://review.openstack.org/293771)

[Steve McLellan (sjmc7)]

Filed https://bugs.launchpad.net/neutron/+bug/1560221 with Neutron. We may not be able to do this because we receive the subnet creation event faster than the DHCP port is created or updated.

[Steve McLellan (sjmc7)]

For now i'm going to disable DHCP port indexing entirely. We can't guarantee we can get information about these ports and it's best given the time in the release to disable their indexing until we can figure out a reliable solution.

[OpenStack Infra (hudson-openstack)]

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

[Travis Tripp (travis-tripp)]

Can you please open a bug / bp for Newton to add DHCP port indexing, so we don't lose track of this after this is closed.

[Steve McLellan (sjmc7)]

https://bugs.launchpad.net/searchlight/+bug/1560608

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/295945
Committed: https://git.openstack.org/cgit/openstack/searchlight/commit/?id=22e05a88a4c68ff2e3d81aadc7b1c965bfb1e80c
Submitter: Jenkins
Branch: master

commit 22e05a88a4c68ff2e3d81aadc7b1c965bfb1e80c
Author: Steve McLellan <email address hidden>
Date: Tue Mar 22 11:04:13 2016 -0500

    Don't index DHCP ports

    We don't have any reliable way to index DHCP ports so for now we won't
    index them on initial indexing; patch also disables their indexing from
    notifications for consistency. If neutron starts sending notifications
    for them both pieces of code should be removed.

    Change-Id: Ia783a9b59fa05b7d89f5add95797fd509ad391a2
    Closes-Bug: #1558790

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/297249

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/297249
Committed: https://git.openstack.org/cgit/openstack/searchlight/commit/?id=b1d339b8da178e87aeda09cae6783b3b6fef504f
Submitter: Jenkins
Branch: stable/mitaka

commit b1d339b8da178e87aeda09cae6783b3b6fef504f
Author: Steve McLellan <email address hidden>
Date: Tue Mar 22 11:04:13 2016 -0500

    Don't index DHCP ports

    We don't have any reliable way to index DHCP ports so for now we won't
    index them on initial indexing; patch also disables their indexing from
    notifications for consistency. If neutron starts sending notifications
    for them both pieces of code should be removed.

    Change-Id: Ia783a9b59fa05b7d89f5add95797fd509ad391a2
    Closes-Bug: #1558790
    (cherry picked from commit 22e05a88a4c68ff2e3d81aadc7b1c965bfb1e80c)

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/searchlight 0.2.0.0rc2 release candidate.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/302579
Committed: https://git.openstack.org/cgit/openstack/searchlight/commit/?id=73b8efe4addac065faa489b5d4ef5dc11ee2706b
Submitter: Jenkins
Branch: master

commit 2a44b637b2ce1202a873cd8d136cc373c3cfc7c6
Author: Lakshmi N Sampath <email address hidden>
Date: Wed Mar 30 20:37:22 2016 -0700

    Backward compatibility with designate v1 api

    Adds backward compatibility for designate v1 api

    dns.record.create, dns.record.update and dns.record.delete
    events needs to be handled if cli user switches between
    v1 and v2 api. Currenlty those are ignored and indexed records
    goes out of sync.

    Also switch dns.domain.xxxxx to dns.zone.xxxxx events since
    zone is v2 api.

    Change-Id: I459841423df776c5afa41255798dacc97b7d9e3e
    Closes-Bug: #1550506

commit 4abf9618720a6c0b2250a92e6869d49dc6aa2032
Author: Steve McLellan <email address hidden>
Date: Tue Mar 29 10:33:02 2016 -0500

    Disable oslo_config file discovery in tests

    oslo_config looks for files in some standard places
    (~/.searchlight.conf, /etc/searchlight/searchlight.conf,
    ./etc/searchlight.conf and possibly more). This means that
    unit tests were using one of these config files which could
    cause failures under some circumstances. This patch monkey
    patches the find_config_files function during tests which
    prevents this behavior.

    Similar to https://review.openstack.org/#/c/172354

    Change-Id: I2b3c98ccd8f3540d5fe8bb1e21596c665600a7f6
    Closes-Bug: #1563053
    (cherry picked from commit 9fecc29a51ac06fa3d27bdd59e5016d77df4d12c)

commit bcc82435ef20edf6fc884b012844f112527161d8
Author: Steve McLellan <email address hidden>
Date: Thu Mar 31 13:50:24 2016 -0500

    Increase heap for elasticsearch in func tests

    We're seeing a number of outofmemory exceptions from elasticsearch.
    Increasing the java heap space to 50m to see if that helps temporarily;
    longer term we need to look at using fewer processes.

    Change-Id: I32277b0613b1ee1a25005655b108fe2b6530679b
    (cherry picked from commit 8a287fcc0d083d133ca4b4d0a75180eeb618ce68)

commit 1e80553aa46153d5a3c0e60047b3d0cc9dbcf5d9
Author: Steve McLellan <email address hidden>
Date: Tue Mar 29 14:27:52 2016 -0500

    Fix _is_multiple_alias_exception signature

    Remove erroneous 'self' from function signature
    to _is_multiple_alias_exception.

    Change-Id: I6fcdd13f360b31be66b553ff8e88011107ecd2b7
    Closes-Bug: #1563516
    (cherry picked from commit 600b691bf950108e93beafe8ae4691503e5b8108)

commit 574a095ebe897cdab935a803ae036c43de32e52f
Author: Steve McLellan <email address hidden>
Date: Mon Mar 28 11:29:54 2016 -0500

    Add missing zero-downtime indexing documentation

    Adds documentation that was not added with zero-downtime reindexing,
    describing the index model and the mechanisms by which plugins are
    indexed.

    Change-Id: Ibd21724d14f2d92a78b908581687ae6811ee3611
    Closes-Bug: #1561190
    (cherry picked from commit a73cd1d85cbf172cfc5882d92d269e1fd5401bdb)

commit bb5ff0701301d6d4b86c313a3d221e7d9c757f75
A...

[Doug Hellmann (doug-hellmann)]

This issue was fixed in the openstack/searchlight 1.0.0.0b1 development milestone.