Merge pull request #52 from signalfx/patched

fix memory leak and upgrade signalfx-python ver
signalfx · Mar 25, 2017 · eac3e0c · eac3e0c
2 parents 593209b + 319327e
commit eac3e0c
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,11 @@
-0.0.29 / 2016-11-02
+0.0.30 / 2017-03-25
 ===================
 
+* Fix memory leak related to cpu utilization metrics collection
+* Upgrade to latest signalfx-python v1.0.16
+
+0.0.29 / 2016-11-02
+
 * Added python dependency six
 
 0.0.28 / 2016-10-18

diff --git a/requirements.txt b/requirements.txt
@@ -1,4 +1,4 @@
 psutil>=3.2.2
-signalfx>=1.0.7
+signalfx>=1.0.16
 simplejson>=3.8.1
 six>=1.10.0
diff --git a/src/signalfx_metadata.py b/src/signalfx_metadata.py
@@ -64,7 +64,7 @@
 TIMEOUT = 3
 POST_URLS = []
 DEFAULT_POST_URL = "https://ingest.signalfx.com/v1/collectd"
-VERSION = "0.0.29"
+VERSION = "0.0.30"
 MAX_LENGTH = 0
 COLLECTD_VERSION = ""
 LINUX_VERSION = ""
@@ -414,6 +414,8 @@ def read(self):
                 else:
                     debug("too old %s %s" % (t, self.metrics[t].keys()))
                     del (self.metrics[t])
+        else:
+            self.metrics = {}
 
 
 class CpuUtilization(Utilization):