OS check in allocation reporter

stackimpact · Oct 2, 2017 · ca08a03 · ca08a03
1 parent 832ee43
commit ca08a03
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 5 deletions.
diff --git a/setup.py b/setup.py
@@ -2,7 +2,7 @@
 
 setup(
     name = 'stackimpact',
-    version = '1.1.2',
+    version = '1.1.3',
     description = 'StackImpact Python Agent',
     author = 'StackImpact',
     author_email = 'devops@stackimpact.com',

diff --git a/stackimpact/agent.py b/stackimpact/agent.py
@@ -26,7 +26,7 @@
 
 class Agent:
 
-    AGENT_VERSION = "1.1.2"
+    AGENT_VERSION = "1.1.3"
     SAAS_DASHBOARD_ADDRESS = "https://agent-api.stackimpact.com"
 
     def __init__(self, **kwargs):

diff --git a/stackimpact/config_loader.py b/stackimpact/config_loader.py
@@ -47,9 +47,11 @@ def load(self):
             if self.agent.config.is_agent_enabled():        
                 self.agent.error_reporter.start()
                 self.agent.process_reporter.start()
+                self.agent.log('Agent activated')
             else:
                 self.agent.error_reporter.stop()
                 self.agent.process_reporter.stop()
+                self.agent.log('Agent deactivated')
 
 
         except Exception:

diff --git a/stackimpact/reporters/allocation_reporter.py b/stackimpact/reporters/allocation_reporter.py
@@ -31,7 +31,7 @@ def start(self):
         if self.agent.get_option('allocation_profiler_disabled'):
             return
 
-        if not min_version(3, 4):
+        if not (runtime_info.OS_LINUX or runtime_info.OS_DARWIN) or not min_version(3, 4):
             self.agent.log('Memory allocation profiling is available for Python 3.4 or higher')
             return
 

diff --git a/tests/reporters/allocation_reporter_test.py b/tests/reporters/allocation_reporter_test.py
@@ -5,13 +5,13 @@
 import threading
 
 import stackimpact
-from stackimpact.runtime import min_version
+from stackimpact.runtime import min_version, runtime_info
 
 
 class AllocationReporterTestCase(unittest.TestCase):
 
     def test_record_allocation_profile(self):
-        if not min_version(3, 4):
+        if not (runtime_info.OS_LINUX or runtime_info.OS_DARWIN) or not min_version(3, 4):
             return
 
         stackimpact._agent = None