Multiple serial update notifications are not buffered

Bug #1436210 reported by Endre Karlson
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Designate
Won't Fix
High
Unassigned

Bug Description

If you create x records this will result in the system attempting to send out y notifications to the downstream dns servers like pdns. In a worse case scenario we might get throttled since we're sending a possible huge amount of consecutive NOTIFY messages.

Maybe these updates should be buffered up before attempting to notify the downstream targets?

Kiall Mac Innes (kiall)
Changed in designate:
milestone: none → liberty-1
importance: Undecided → Medium
status: New → Confirmed
Kiall Mac Innes (kiall)
Changed in designate:
status: Confirmed → Triaged
importance: Medium → High
Thierry Carrez (ttx)
Changed in designate:
milestone: liberty-1 → liberty-2
sonu (sonu-bhumca11)
Changed in designate:
assignee: nobody → sonu (sonu-bhumca11)
Kiall Mac Innes (kiall)
Changed in designate:
milestone: liberty-2 → none
Changed in designate:
assignee: sonu (sonu-bhumca11) → Federico Ceratto (federico-ceratto)
status: Triaged → In Progress
Revision history for this message
Federico Ceratto (federico-ceratto) wrote :

https://review.openstack.org/#/c/247588/ implements a delayed notification mechanism.

The next step would be to update zone creation/update/deletion to make use of delayed notify.

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

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

commit ecda910c2be4e9c733abf31eb34b1f624a7647ce
Author: Federico Ceratto <email address hidden>
Date: Thu Nov 19 15:21:11 2015 +0000

    Add delayed NOTIFY

    Based on specs in https://review.openstack.org/#/c/246515
    Add database column zones.pending_notify and migration script
    Add Zone Manager Task to perform delayed NOTIFY
    Add Admin API method to count zones pending notify

    closes-bug: #1498462
    partial-bug: #1436210
    Change-Id: Ic5c2745bcd2cc057a42cb8edf21ed4c1a3ddc952
    Depends-On: If15594099eb7cf74f1c534a05884c2d2501e57e6

Changed in designate:
status: In Progress → Triaged
assignee: Federico Ceratto (federico-ceratto) → nobody
Revision history for this message
Graham Hayes (grahamhayes) wrote :

This has been open for a long period of time, with a low priority.

As a result, I am closing the bug.

If you feel this was in error, and this is important please re-open, and feel free to start working on the bug.

Changed in designate:
status: Triaged → Won't Fix
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.