From: Ryan Moats <rmoats@us.ibm.com>
Date: Fri, 11 Sep 2015 12:41:38 +0000 (-0500)
Subject: Remove useless log from periodic_sync_routers_task
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=fb8014294530ac83f209a79612c09c897d80227f;p=openstack-build%2Fneutron-build.git

Remove useless log from periodic_sync_routers_task

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>
---

diff --git a/neutron/agent/l3/agent.py b/neutron/agent/l3/agent.py
index 99921846c..1927ab7c6 100644
--- a/neutron/agent/l3/agent.py
+++ b/neutron/agent/l3/agent.py
@@ -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