From afe51251638ae91766c82edc299812c09b7da636 Mon Sep 17 00:00:00 2001
From: Julien Fortin <julien@cumulusnetworks.com>
Date: Thu, 12 May 2016 16:00:41 +0200
Subject: [PATCH] Fixing errors and typos

Ticket: None
Reviewed By: Roopa
Testing Done:

Signed-off-by: Julien Fortin <julien@cumulusnetworks.com>
---
 addons/bridge.py           | 8 ++++----
 ifupdown/graph.py          | 6 ++----
 ifupdown/scheduler.py      | 2 +-
 ifupdown/statemanager.py   | 3 ++-
 ifupdownaddons/bondutil.py | 3 +--
 5 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/addons/bridge.py b/addons/bridge.py
index ac3686c..2614256 100644
--- a/addons/bridge.py
+++ b/addons/bridge.py
@@ -1315,8 +1315,8 @@ class bridge(moduleBase):
             running_bridge_port_vids = ''
             portlist = self.parse_port_list(ifaceobj.name, attrval)
             if not portlist:
-                self.log_warn('%s: could not parse \'%s %s\''
-                          %(ifaceobj.name, attrname, attrval))
+                self.log_warn('%s: could not parse \'bridge-port-vids %s\''
+                          %(ifaceobj.name, attrval))
                 return
             err = 0
             for p in portlist:
@@ -1347,8 +1347,8 @@ class bridge(moduleBase):
         if attrval:
             portlist = self.parse_port_list(ifaceobj.name, attrval)
             if not portlist:
-                self.log_warn('%s: could not parse \'%s %s\''
-                              %(ifaceobj.name, attrname, attrval))
+                self.log_warn('%s: could not parse \'bridge-port-pvids %s\''
+                              %(ifaceobj.name, attrval))
                 return
             running_bridge_port_pvids = ''
             err = 0
diff --git a/ifupdown/graph.py b/ifupdown/graph.py
index 23882ff..e1729d0 100644
--- a/ifupdown/graph.py
+++ b/ifupdown/graph.py
@@ -18,9 +18,7 @@ except ImportError, e:
 class graph():
     """ graph functions to sort and print interface graph """
 
-    def __init__(self):
-        self.logger = logging.getLogger('ifupdown.' +
-                    self.__class__.__name__)
+    logger = logging.getLogger('ifupdown.graph')
 
     @classmethod
     def topological_sort_graphs_all(cls, dependency_graphs, indegrees_arg):
@@ -54,7 +52,7 @@ class graph():
                 try:
                     indegrees[y] = indegrees.get(y) - 1
                 except:
-                    self.logger.debug('topological_sort_graphs_all: did not find %s' %y)
+                    cls.logger.debug('topological_sort_graphs_all: did not find %s' %y)
                     indegrees[y] = 0
                     pass
                 if indegrees.get(y) == 0:
diff --git a/ifupdown/scheduler.py b/ifupdown/scheduler.py
index 1d0d92d..d906a11 100644
--- a/ifupdown/scheduler.py
+++ b/ifupdown/scheduler.py
@@ -408,7 +408,7 @@ class ifaceScheduler():
                 cls.run_iface_list_ops(ifupdownobj, ifaceobjs, ops)
             except Exception, e:
                 if continueonfailure:
-                   self.logger.warn('%s' %str(e))
+                    ifupdownobj.logger.warn('%s' %str(e))
 
     @classmethod
     def get_sorted_iface_list(cls, ifupdownobj, ifacenames, ops,
diff --git a/ifupdown/statemanager.py b/ifupdown/statemanager.py
index a7d75a0..9eee24f 100644
--- a/ifupdown/statemanager.py
+++ b/ifupdown/statemanager.py
@@ -9,6 +9,7 @@
 import cPickle
 from collections import OrderedDict
 import logging
+import exceptions
 import os
 from iface import *
 
@@ -177,7 +178,7 @@ class stateManager():
             for i in ifacenames:
                 ifaceobj = self.ifaces.get(i)
                 if ifaceobj is None:
-                    raise ifaceNotFoundError('ifname %s'
+                    raise exceptions.ifaceNotFoundError('ifname %s'
                         %i + ' not found')
                 ifaceobj.dump(self.logger)
         else:
diff --git a/ifupdownaddons/bondutil.py b/ifupdownaddons/bondutil.py
index 59750cd..823b911 100644
--- a/ifupdownaddons/bondutil.py
+++ b/ifupdownaddons/bondutil.py
@@ -241,8 +241,7 @@ class bondutil(utilsBase):
         return self._cache_get([bondname, 'linkinfo', 'lacp_rate'])
 
     def set_lacp_bypass_allow(self, bondname, allow, prehook=None, posthook=None):
-        if (self._cache_check([bondname, 'linkinfo', 'lacp_bypass'],
-                lacp_bypass)):
+        if (self._cache_check([bondname, 'linkinfo', 'lacp_bypass'], allow)):
             return
         if prehook:
             prehook(bondname)