sync-domains generates 504 error due MessagingTimeout error

Bug #1390370 reported by Pavel Kirpichyov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Designate
Won't Fix
Low
Abhilash Goyal

Bug Description

If designate has a lot of domains with records sync-domains command will throw MessagingTimeout.

api extension calls sync-domains method by rpc: https://github.com/openstack/designate/blob/master/designate/api/v1/extensions/sync.py#L31
and waiting result... but does not handle it

As option to fix problem it possible to use rpc cast method instead of call and return 202 HTTP response code.

Kiall Mac Innes (kiall)
Changed in designate:
status: New → Confirmed
importance: Undecided → High
Revision history for this message
Graham Hayes (grahamhayes) wrote :

we have a possible problem here...

to fix this, we would have to move the operation to an async method, which would be a breaking change in the API....

Revision history for this message
Kiall Mac Innes (kiall) wrote :

I think, as an admin only API with a real problem, we can do this.. That said.. I'd prefer to remove these methods entirely as the pools periodic functions are effectively doing just this..

Revision history for this message
Federico Ceratto (federico-ceratto) wrote :

The bug affects the V1 API and it does not affect newer releases.

Changed in designate:
importance: High → Low
Changed in designate:
assignee: nobody → Abhilash Goyal (abhilash-goyal)
Revision history for this message
Abhilash Goyal (abhilash-goyal) wrote :

Hello Pavel,
Your bug description says 'lot of domains', is there any finite number above which MessagingTimeout will be thrown. Kindly provide steps to replicate this bug.

Revision history for this message
Pavel Kirpichyov (pavel-kirpichyov) wrote :

Hello!
Unfortunately, I had finished my work with openstack and designate.
As I remember it was about 2000 domains in designate with powerdns as backend

Revision history for this message
Graham Hayes (grahamhayes) wrote :

This is for an old version of our API. sync is now depricated

Changed in designate:
status: Confirmed → 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.