From: Thierry Parmentelat <thierry.parmentelat@sophia.inria.fr>
Date: Thu, 15 Dec 2011 08:50:54 +0000 (+0100)
Subject: Merge branch 'master' of ssh://git.onelab.eu/git/sfa
X-Git-Tag: sfa-2.0-6~7
X-Git-Url: http://git.onelab.eu/?p=sfa.git;a=commitdiff_plain;h=0a9aae617f7d6ceb4e169fedc6883928dce0b7b6;hp=e95161b0e9ec90bc71d61b8c6a10db020fe2ff90

Merge branch 'master' of ssh://git.onelab.eu/git/sfa
---

diff --git a/cron.d/sfa.cron b/cron.d/sfa.cron
index c92afb6d..00d27b6c 100644
--- a/cron.d/sfa.cron
+++ b/cron.d/sfa.cron
@@ -1,8 +1,14 @@
+#
+# this example file should be installed in /etc/cron.d owned by root
+#
 SHELL=/bin/bash
 PATH=/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/bin
+# xxx replace this
 MAILTO=server-msgs@planet-lab.org
 HOME=/
 #
 # minute hour day-of-month month day-of-week user command
-0 * * * *  /usr/bin/sfa-import-plc.py > /dev/null 2>&1
-0 0 * * * /usr/bin/sfa-clean-peer-records.py > /dev/null 2>&1  
+# once or twice an hour makes sense
+0 * * * * root /usr/bin/sfa-import-plc.py         >> /var/log/sfa_import.log 2>&1
+# this is needed only if you run RefreshPeer
+#0 0 * * * root /usr/bin/sfa-clean-peer-records.py >> /var/log/sfa_import.log 2>&1
diff --git a/sfa/generic/__init__.py b/sfa/generic/__init__.py
index 3c3855dc..de1a9e14 100644
--- a/sfa/generic/__init__.py
+++ b/sfa/generic/__init__.py
@@ -60,6 +60,8 @@ class Generic:
         if not 'interface' in kwargs:
             logger.critical("Generic.make_api: no interface found")
         api = self.api_class()(*args, **kwargs)
+        # xxx can probably drop support for managers implemented as modules 
+        # which makes it a bit awkward
         manager_class_or_module = self.make_manager(api.interface)
         driver = self.make_driver (api.config, api.interface)
         ### arrange stuff together