Action: store a logger object in each Action.

This will allow log messages to include the action they originate from
in the future.

diff --git a/src/lib/backupninja/handlers/__init__.py b/src/lib/backupninja/handlers/__init__.py
index 29dc75631823ed79d5ecda18b99698d6dcacb782..484677171e0e72ec57250bbdfeaf3b554acf226e 100644 (file)
--- a/src/lib/backupninja/handlers/__init__.py
+++ b/src/lib/backupninja/handlers/__init__.py
@@ -55,6 +55,10 @@ class Action(object):
         # Assume we'll run succesfully. If anything fails in the
         # meanwhile, set this to True.
         self.failed = False
         # Assume we'll run succesfully. If anything fails in the
         # meanwhile, set this to True.
         self.failed = False

+        # A logger object for this action. In the future, this might
+        # become a specific logger, that includes the action name and
+        # type.
+        self.log = log

 
     def run(self, **kwargs):
         """
 
     def run(self, **kwargs):
         """

diff --git a/src/lib/backupninja/handlers/test.py b/src/lib/backupninja/handlers/test.py
index 3b8c5d42640da0eca49bbe2aec59b2c474780afd..33a4f47d11ce535b9e293a08a489293fd17e9031 100644 (file)
--- a/src/lib/backupninja/handlers/test.py
+++ b/src/lib/backupninja/handlers/test.py
@@ -21,12 +21,10 @@
 
 """ Testing handler that logs a configurable message """
 
 
 """ Testing handler that logs a configurable message """
 

-import logging as log
-

 from backupninja.handlers import Action
 
 class TestAction(Action):
     def run(self, **kwargs):
 from backupninja.handlers import Action
 
 class TestAction(Action):
     def run(self, **kwargs):

-        log.info(self.conf.get('main', 'message'))
+        self.log.info(self.conf.get('main', 'message'))

         
 handler = TestAction
         
 handler = TestAction