Deleting Record With BIND 9 Driver Fails

In designate/storage/api.py, we have:

@contextlib.contextmanager
    def delete_record(self, context, record_id):
        """
Delete a record

:param context: RPC Context
:param record_id: Record ID to delete
"""
        yield self.storage.get_record(context, record_id)
        self.storage.delete_record(context, record_id)

If we change this to:

@contextlib.contextmanager
    def delete_record(self, context, record_id):
        """
Delete a record

:param context: RPC Context
:param record_id: Record ID to delete
"""
        record = self.storage.get_record(context, record_id)
        self.storage.delete_record(context, record_id)
        yield record

On the surface, it fixes the BIND 9 driver and we can delete records successfully. I'm guessing this is not desired however. Am I correct about this?

The real issue here is how the backend BIND 9 driver works. The code as written gets the record, passes the record to the backend, and the backend should do something with that record. With the BIND 9 driver, it doesn't do anything with the record, but constructs a new zone file and reloads it. Then the record is delete from the central storage. Unfortunately, the record was present during the zone file creation, hence the record is not being deleted.

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

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

Reviewed: https://review.openstack.org/96204
Committed: https://git.openstack.org/cgit/stackforge/designate/commit/?id=a4cb85e11d774b3b49d138baa6d47e4cb36a4789
Submitter: Jenkins
Branch: master

commit a4cb85e11d774b3b49d138baa6d47e4cb36a4789
Author: rjrjr <email address hidden>
Date: Fri May 30 19:52:40 2014 -0700

    Bug #1261894: Deleting Record With BIND 9 Driver Fails

    Fix the delete_record/delete_recordset method so it deletes the
    record/recordset in the central storage before the BIND 9 backend
    gets the request to delete the record/recordset.

    Changed the other delete_<object> methods (quota, server, etc.) with the
    same fix. Finally, all the delete_<object> methods in the storage have
    been simplified and make less SQL calls by returning the object from the
    delete_<object> method instead of making another get_<object> call.

    Change-Id: I10ceb1f5c2550724f1a752a79319a4bbf366876b
    Closes-bug: #1261894