During DhcpAgent startup procedure all the following networks
initialization is actually perform twice:
* Killing old dnsmasq processes
* set and configure all TAP interfaces
* building all Dnsmasq config files (lease and host files)
* launching dnsmasq processes
What is done during the second iteration is just clean and redo
exactly the same another time! This is really inefficient and
increase dramatically DHCP startup time (near twice than needed).
Initialization process 'sync_state' method is called twice:
* one time during init_host()
* another time during _report_state()
sync_state() call must stay in init_host() due to bug #1420042.
sync_state() is always called during startup in init_host()
and will be periodically called by periodic_resync()
to do reconciliation.
Hence it can safely be removed from the run() method.
Change-Id: Id6433598d5c833d2e86be605089d42feee57c257
Closes-bug: #1651368
Closes-Bug: #1650611
(cherry picked from commit f15851b98974dc16606da195cf3ecee577cd0ef8)
(cherry picked from commit ae0a31d8b0c449031432117ac33c1e5bdf9d5957)
Reviewed: https:/ /review. openstack. org/423206 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=21963618ca7 accfe0b5d3389ee 15426d5032a2ac
Committed: https:/
Submitter: Jenkins
Branch: stable/mitaka
commit 21963618ca7accf e0b5d3389ee1542 6d5032a2ac
Author: Bertrand Lallau <email address hidden>
Date: Tue Dec 20 10:53:41 2016 +0100
DHCP: enhance DHCPAgent startup procedure
During DhcpAgent startup procedure all the following networks
initialization is actually perform twice:
* Killing old dnsmasq processes
* set and configure all TAP interfaces
* building all Dnsmasq config files (lease and host files)
* launching dnsmasq processes
What is done during the second iteration is just clean and redo
exactly the same another time! This is really inefficient and
increase dramatically DHCP startup time (near twice than needed).
Initialization process 'sync_state' method is called twice:
* one time during init_host()
* another time during _report_state()
sync_state() call must stay in init_host() due to bug #1420042.
sync_state() is always called during startup in init_host()
and will be periodically called by periodic_resync()
to do reconciliation.
Hence it can safely be removed from the run() method.
Change-Id: Id6433598d5c833 d2e86be605089d4 2feee57c257 6606da195cf3ece e577cd0ef8) 31432117ac33c1e 5bdf9d5957)
Closes-bug: #1651368
Closes-Bug: #1650611
(cherry picked from commit f15851b98974dc1
(cherry picked from commit ae0a31d8b0c4490