Designate record deletion using wrong event name

Bug #1550506 reported by Steve McLellan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Searchlight
Fix Released
High
Lakshmi N Sampath

Bug Description

It looks like deletions on designate records issues a 'dns.record.delete' event, not 'dns.recordset.delete' (which i haven't seen happen at all). There is also dns.record.create in addition to dns.recordset.create, and dns.zone equivalents of all the dns.domain but i don't know if we need to track them; perhaps follow up with the designate team.

Revision history for this message
Steve McLellan (sjmc7) wrote :

Looking at the designate source (designate/central/service.py) there are notifications for dns.recordset and dns.record; we may need to handle both. Need to talk to the designate team.

Neha (neha-m-pande)
Changed in searchlight:
assignee: nobody → Neha (neha-m-pande)
Changed in searchlight:
milestone: none → mitaka-rc1
importance: Undecided → High
Revision history for this message
Travis Tripp (travis-tripp) wrote :

Hi Neha, are you going to be able to put up a patch on this in the next week?

Revision history for this message
Neha (neha-m-pande) wrote :

I haven't started working on this.

Changed in searchlight:
assignee: Neha (neha-m-pande) → nobody
Changed in searchlight:
assignee: nobody → Lakshmi N Sampath (lakshmi-sampath)
Changed in searchlight:
milestone: mitaka-rc1 → mitaka-rc2
Changed in searchlight:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to searchlight (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to searchlight (stable/mitaka)

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

Changed in searchlight:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to searchlight (master)

Reviewed: https://review.openstack.org/299733
Committed: https://git.openstack.org/cgit/openstack/searchlight/commit/?id=3b7dc517a27ca3bc002ae7006367f4bfced7c122
Submitter: Jenkins
Branch: master

commit 3b7dc517a27ca3bc002ae7006367f4bfced7c122
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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to searchlight (stable/mitaka)

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

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

tags: added: in-stable-mitaka
Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/searchlight 0.2.0.0rc2

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to searchlight (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to searchlight (master)
Download full text (5.8 KiB)

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...

Read more...

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/searchlight 1.0.0.0b1

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

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.