bgp logging fixes

Change-Id: I805ed00d95f2700e30b363f4eaf1067abeab22ab
Signed-off-by: Sam Hague <shague@redhat.com>

diff --git a/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/BgpConfigurationManager.java b/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/BgpConfigurationManager.java
index a4a68a21b4da3306ad489e4ff4f4a9e4d67ed986..6e4ab8360042f89210c69c4fd34834c7b4108f1c 100644 (file)
--- a/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/BgpConfigurationManager.java
+++ b/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/BgpConfigurationManager.java
@@ -322,7 +322,7 @@ public class BgpConfigurationManager {
                         activateMIP();
                         bgpRestarted();
                     } else {
-                        LOG.error("Not owner: hasOwner: {}, isOwner: {}",ownershipChange.hasOwner(),
+                        LOG.info("Not owner: hasOwner: {}, isOwner: {}",ownershipChange.hasOwner(),
                                 ownershipChange.isOwner() );
                     }
                 }
@@ -1389,26 +1389,26 @@ public class BgpConfigurationManager {
             @Override
             public void run() {
                 try {
-                    LOG.error("running bgp replay task ");
+                    LOG.info("running bgp replay task ");
                     if (get() == null) {
                         String host = getConfigHost();
                         int port = getConfigPort();
                         LOG.info("connecting  to bgp host {} ", host);
 
                         boolean res = bgpRouter.connect(host, port);
-                        LOG.error("no config to push in bgp replay task ");
+                        LOG.info("no config to push in bgp replay task ");
                         return;
                     }
                     setStaleStartTime(System.currentTimeMillis());
-                    LOG.error("started creating stale fibDSWriter  map ");
+                    LOG.info("started creating stale fibDSWriter  map ");
                     createStaleFibMap();
                     setStaleEndTime(System.currentTimeMillis());
-                    LOG.error("took {} msecs for stale fibDSWriter map creation ", getStaleEndTime()- getStaleStartTime());
-                    LOG.error("started bgp config replay ");
+                    LOG.info("took {} msecs for stale fibDSWriter map creation ", getStaleEndTime()- getStaleStartTime());
+                    LOG.info("started bgp config replay ");
                     setCfgReplayStartTime(System.currentTimeMillis());
                     replay();
                     setCfgReplayEndTime(System.currentTimeMillis());
-                    LOG.error("took {} msecs for bgp replay ", getCfgReplayEndTime() - getCfgReplayStartTime());
+                    LOG.info("took {} msecs for bgp replay ", getCfgReplayEndTime() - getCfgReplayStartTime());
                     long route_sync_time = getStalePathtime(BGP_RESTART_ROUTE_SYNC_SEC, config.getAsId());
                     Thread.sleep(route_sync_time * 1000L);
                     setStaleCleanupTime(route_sync_time);

diff --git a/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/thrift/server/BgpThriftService.java b/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/thrift/server/BgpThriftService.java
index b25bb05eea1a06e0e90bee308bf9a72c783fef0c..575c9751823a8b2a7ff07854677b0b3d7baf3d8c 100644 (file)
--- a/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/thrift/server/BgpThriftService.java
+++ b/vpnservice/bgpmanager/bgpmanager-impl/src/main/java/org/opendaylight/netvirt/bgpmanager/thrift/server/BgpThriftService.java
@@ -75,19 +75,19 @@ public class BgpThriftService {
                 server.setServerEventHandler(new TServerEventHandler() {
                     @Override
                     public void preServe() {
-                        LOGGER.error("Bgp thrift server pre serve event");
+                        LOGGER.info("Bgp thrift server pre serve event");
                     }
 
                     @Override
                     public ServerContext createContext(TProtocol input, TProtocol output) {
-                        LOGGER.error("Bgp thrift server create context event");
+                        LOGGER.info("Bgp thrift server create context event");
                         synchronized (this) {
                             try {
                                 if (oldThriftClientContext != null) {
-                                    LOGGER.error("Bgp thrift server closing old context");
+                                    LOGGER.info("Bgp thrift server closing old context");
                                     oldThriftClientContext.getIn().getTransport().close();
                                 } else {
-                                    LOGGER.error("Bgp thrift server old context is null nothing to close");
+                                    LOGGER.info("Bgp thrift server old context is null nothing to close");
                                 }
                                 oldThriftClientContext = null;
                             } catch (Throwable ignore) {
@@ -99,12 +99,12 @@ public class BgpThriftService {
 
                     @Override
                     public void deleteContext(ServerContext serverContext, TProtocol input, TProtocol output) {
-                        LOGGER.error("Bgp thrift server delete context event");
+                        LOGGER.info("Bgp thrift server delete context event");
                         if (oldThriftClientContext == serverContext) {
-                            LOGGER.error("Bgp thrift server cleanup old context");
+                            LOGGER.info("Bgp thrift server cleanup old context");
                             oldThriftClientContext = null;
                         } else {
-                            LOGGER.error("Bgp thrift server cleanup context");
+                            LOGGER.info("Bgp thrift server cleanup context");
                         }
                     }
 
@@ -140,7 +140,7 @@ public class BgpThriftService {
         }
 
         public void onStartConfigResyncNotification() {
-            LOGGER.error("BGP (re)started");
+            LOGGER.info("BGP (re)started");
             bgpManager.setqBGPrestartTS(System.currentTimeMillis());
             try {
                 bgpManager.bgpRestarted();