From 748a6c81e4233d7b4c75d9a529be26ada4306c5b Mon Sep 17 00:00:00 2001
From: "Chris St. Pierre" <chris.a.st.pierre@gmail.com>
Date: Fri, 29 Mar 2013 19:05:07 -0400
Subject: Added option to periodically dump performance stats to logs

---
 src/lib/Bcfg2/Options.py     | 13 ++++++++++++-
 src/lib/Bcfg2/Server/Core.py | 24 +++++++++++++++++++++++-
 2 files changed, 35 insertions(+), 2 deletions(-)

diff --git a/src/lib/Bcfg2/Options.py b/src/lib/Bcfg2/Options.py
index 66e987b91..099709cbc 100644
--- a/src/lib/Bcfg2/Options.py
+++ b/src/lib/Bcfg2/Options.py
@@ -1082,6 +1082,15 @@ VERBOSE = \
            cmd='-v',
            cook=get_bool,
            cf=('logging', 'verbose'))
+LOG_PERFORMANCE = \
+    Option("Periodically log performance statistics",
+           default=False,
+           cf=('logging', 'performance'))
+PERFLOG_INTERVAL = \
+    Option("Performance statistics logging interval in seconds",
+           default=300.0,
+           cook=get_timeout,
+           cf=('logging', 'performance_interval'))
 
 # Plugin-specific options
 CFG_VALIDATION = \
@@ -1164,7 +1173,9 @@ SERVER_COMMON_OPTIONS = dict(repo=SERVER_REPOSITORY,
                              web_configfile=WEB_CFILE,
                              backend=SERVER_BACKEND,
                              vcs_root=SERVER_VCS_ROOT,
-                             authentication=SERVER_AUTHENTICATION)
+                             authentication=SERVER_AUTHENTICATION,
+                             perflog=LOG_PERFORMANCE,
+                             perflog_interval=PERFLOG_INTERVAL)
 
 CRYPT_OPTIONS = dict(encrypt=ENCRYPT,
                      decrypt=DECRYPT,
diff --git a/src/lib/Bcfg2/Server/Core.py b/src/lib/Bcfg2/Server/Core.py
index deb9065a5..a0044d561 100644
--- a/src/lib/Bcfg2/Server/Core.py
+++ b/src/lib/Bcfg2/Server/Core.py
@@ -96,6 +96,7 @@ class BaseCore(object):
         .. automethod:: _block
         .. -----
         .. automethod:: _file_monitor_thread
+        .. automethod:: _perflog_thread
         """
         #: The Bcfg2 repository directory
         self.datastore = setup['repo']
@@ -317,6 +318,12 @@ class BaseCore(object):
             threading.Thread(name="%sFAMThread" % setup['filemonitor'],
                              target=self._file_monitor_thread)
 
+        self.perflog_thread = None
+        if self.setup['perflog']:
+            self.perflog_thread = \
+                threading.Thread(name="PerformanceLoggingThread",
+                                 target=self._perflog_thread)
+
         #: A :func:`threading.Lock` for use by
         #: :func:`Bcfg2.Server.FileMonitor.FileMonitor.handle_event_set`
         self.lock = threading.Lock()
@@ -349,11 +356,23 @@ class BaseCore(object):
                        if isinstance(plugin, base_cls)],
                       key=lambda p: (p.sort_order, p.name))
 
+    def _perflog_thread(self):
+        """ The thread that periodically logs performance statistics
+        to syslog. """
+        self.logger.debug("Performance logging thread starting")
+        while not self.terminate.isSet():
+            self.terminate.wait(self.setup['perflog_interval'])
+            for name, stats in self.get_statistics(None).items():
+                self.logger.info("Performance statistics: "
+                                 "%s min=%.06f, max=%.06f, average=%.06f, "
+                                 "count=%d" % ((name, ) + stats))
+
     def _file_monitor_thread(self):
         """ The thread that runs the
         :class:`Bcfg2.Server.FileMonitor.FileMonitor`. This also
         queries :class:`Bcfg2.Server.Plugin.interfaces.Version`
         plugins for the current revision of the Bcfg2 repo. """
+        self.logger.debug("File monitor thread starting")
         famfd = self.fam.fileno()
         terminate = self.terminate
         while not terminate.isSet():
@@ -718,7 +737,8 @@ class BaseCore(object):
         :type event: Bcfg2.Server.FileMonitor.Event
         """
         if event.filename != self.cfile:
-            print("Got event for unknown file: %s" % event.filename)
+            self.logger.error("Got event for unknown file: %s" %
+                              event.filename)
             return
         if event.code2str() == 'deleted':
             return
@@ -758,6 +778,8 @@ class BaseCore(object):
             self.fam.start()
             self.fam_thread.start()
             self.fam.AddMonitor(self.cfile, self)
+            if self.perflog_thread is not None:
+                self.perflog_thread.start()
 
             for plug in self.plugins_by_type(Bcfg2.Server.Plugin.Threaded):
                 plug.start_threads()
-- 
cgit v1.2.3-1-g7c22