commit 8c989e67b99745f55d462e21be0eaa00f6a0e9b8
Author: Kevin Benton <email address hidden>
Date: Thu Jan 15 08:11:49 2015 -0800
Make L3 agent honor periodic_interval setting
The periodic_task decorator for the sync routers task was resulting
in a default spacing of 60 seconds. This meant that any values less
than that for the periodic_interval setting would not work correctly.
The fixed interval looping call would run at the periodic_interval
but this task would not execute every time as expected.
For example, if the periodic_interval was 40 seconds, the task would
only end up running every other interval (80 seconds in this case)
because every other attempt would be blocked by the default 60 second
barrier of periodic_task.
This sets the periodic_taks spacing variable to 1 second so the
interval is controlled only by the loopingcall as expected.
Ultimately periodic_task should probably be completely removed since
it's not compatible with the fixed interval loopingcall in this manner.
Reviewed: https:/ /review. openstack. org/147710 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=8c989e67b99 745f55d462e21be 0eaa00f6a0e9b8
Committed: https:/
Submitter: Jenkins
Branch: master
commit 8c989e67b99745f 55d462e21be0eaa 00f6a0e9b8
Author: Kevin Benton <email address hidden>
Date: Thu Jan 15 08:11:49 2015 -0800
Make L3 agent honor periodic_interval setting
The periodic_task decorator for the sync routers task was resulting
in a default spacing of 60 seconds. This meant that any values less
than that for the periodic_interval setting would not work correctly.
The fixed interval looping call would run at the periodic_interval
but this task would not execute every time as expected.
For example, if the periodic_interval was 40 seconds, the task would
only end up running every other interval (80 seconds in this case)
because every other attempt would be blocked by the default 60 second
barrier of periodic_task.
This sets the periodic_taks spacing variable to 1 second so the
interval is controlled only by the loopingcall as expected.
Ultimately periodic_task should probably be completely removed since
it's not compatible with the fixed interval loopingcall in this manner.
Closes-Bug: #1411085 b241692f00f9b5a 2f923e3cf31
Change-Id: I23818c3fab2640