]> git.puffer.fish Git - mirror/frr.git/commitdiff
topotest/topogen: demote some messages level
authorRafael Zalamena <rzalamena@opensourcerouting.org>
Wed, 8 Nov 2017 18:23:14 +0000 (16:23 -0200)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Wed, 28 Nov 2018 01:22:12 +0000 (20:22 -0500)
The messages level of this commit were downgraded because some of them
happen on 'non-error' situations. This should help diminish the error
log verbosity on the CI-system run.

tests/topotests/lib/topogen.py
tests/topotests/lib/topotest.py

index 8e0d68e86d61b07fa43b88a41b2437ba1681bf2e..43a886befb037385ddfe77037c53b92af2170be8 100644 (file)
@@ -354,7 +354,7 @@ class Topogen(object):
 
     def set_error(self, message, code=None):
         "Sets an error message and signal other tests to skip."
-        logger.error(message)
+        logger.info(message)
 
         # If no code is defined use a sequential number
         if code is None:
index ccdea83393ed61d5ed31d716f558d548b91ccb9d..8f456846c0fc6980162a26fdc9992e2d5c54df77 100644 (file)
@@ -566,7 +566,7 @@ class Router(Node):
             self.cmd('chown %s:%s /etc/%s/%s.conf' % (self.routertype, self.routertype, self.routertype, daemon))
             self.waitOutput()
         else:
-            logger.warning('No daemon {} known'.format(daemon))
+            logger.info('No daemon {} known'.format(daemon))
         # print "Daemons after:", self.daemons
     def startRouter(self):
         # Disable integrated-vtysh-config
@@ -584,20 +584,20 @@ class Router(Node):
         if self.daemons['ldpd'] == 1:
             ldpd_path = os.path.join(self.daemondir, 'ldpd')
             if not os.path.isfile(ldpd_path):
-                logger.warning("LDP Test, but no ldpd compiled or installed")
+                logger.info("LDP Test, but no ldpd compiled or installed")
                 return "LDP Test, but no ldpd compiled or installed"
 
             if version_cmp(platform.release(), '4.5') < 0:
-                logger.warning("LDP Test need Linux Kernel 4.5 minimum")
+                logger.info("LDP Test need Linux Kernel 4.5 minimum")
                 return "LDP Test need Linux Kernel 4.5 minimum"
 
             # Check if required kernel modules are available with a dryrun modprobe
             # Silent accept of modprobe command assumes ok status
             if self.cmd('/sbin/modprobe -n mpls-router' ) != "":
-                logger.warning("LDP Test needs mpls-router kernel module")
+                logger.info("LDP Test needs mpls-router kernel module")
                 return "LDP Test needs mpls-router kernel module"
             if self.cmd('/sbin/modprobe -n mpls-iptunnel') != "":
-                logger.warning("LDP Test needs mpls-iptunnel kernel module")
+                logger.info("LDP Test needs mpls-iptunnel kernel module")
                 return "LDP Test needs mpls-router kernel module"
 
             self.cmd('/sbin/modprobe mpls-router')
@@ -607,7 +607,7 @@ class Router(Node):
         if self.daemons['eigrpd'] == 1:
             eigrpd_path = os.path.join(self.daemondir, 'eigrpd')
             if not os.path.isfile(eigrpd_path):
-                logger.warning("EIGRP Test, but no eigrpd compiled or installed")
+                logger.info("EIGRP Test, but no eigrpd compiled or installed")
                 return "EIGRP Test, but no eigrpd compiled or installed"
 
         # Init done - now restarting daemons