]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Remove useless log from periodic_sync_routers_task
authorRyan Moats <rmoats@us.ibm.com>
Fri, 11 Sep 2015 12:41:38 +0000 (07:41 -0500)
committerRyan Moats <rmoats@us.ibm.com>
Fri, 11 Sep 2015 12:41:38 +0000 (07:41 -0500)
Logging that peridoic_sync_routers_task is starting with fullsync
False just adds noise to devstack logs.  Reposition the log
statement to indicate that the task is starting if it is going
to be doing real processing.

Change-Id: I73def1e20218b01c135769d0b8fbce449dad17ea
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>
neutron/agent/l3/agent.py

index 99921846c3e06680d7787f73302f878e3346cae6..1927ab7c6854655c8382b453b551887d3f817e8c 100644 (file)
@@ -517,10 +517,9 @@ class L3NATAgent(firewall_l3_agent.FWaaSL3AgentRpcCallback,
     @periodic_task.periodic_task(spacing=1)
     def periodic_sync_routers_task(self, context):
         self.process_services_sync(context)
-        LOG.debug("Starting periodic_sync_routers_task - fullsync:%s",
-                  self.fullsync)
         if not self.fullsync:
             return
+        LOG.debug("Starting fullsync periodic_sync_routers_task")
 
         # self.fullsync is True at this point. If an exception -- caught or
         # uncaught -- prevents setting it to False below then the next call