feat(profiler): configurable profile window length

RisingStack · May 9, 2017 · 02a4e1d · 02a4e1d
1 parent fc78970
commit 02a4e1d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/lib/agent/profiler/cpu/index.js b/lib/agent/profiler/cpu/index.js
@@ -10,7 +10,7 @@ function CpuProfiler (options) {
   var self = this
   this.collectorApi = options.collectorApi
   this.controlBus = options.controlBus
-  this.profileWindow = 10000
+  this.profileWindow = options.config.profilerWindowLength
 
   this.controlBus.on('cpu-profile', function (command) {
     self.sendProfile(command)

diff --git a/lib/agent/profiler/cpu/index.spec.js b/lib/agent/profiler/cpu/index.spec.js
@@ -34,7 +34,8 @@ describe('The Cpu Profiler module', function () {
 
     var profiler = CpuProfiler.create({
       collectorApi: collectorApi,
-      controlBus: msgBus
+      controlBus: msgBus,
+      config: {}
     })
 
     profiler.profileWindow = 0

diff --git a/lib/utils/configReader.js b/lib/utils/configReader.js
@@ -64,7 +64,10 @@ ConfigReader.prototype._getEnvVarConfig = function () {
       : undefined,
     proxy: process.env.TRACE_PROXY || process.env.HTTPS_PROXY || process.env.https_proxy || process.env.HTTP_PROXY,
     keepQueryParams: process.env.TRACE_KEEP_QUERY_PARAMS,
-    collectSeverity: process.env.TRACE_COLLECT_SEVERITY
+    collectSeverity: process.env.TRACE_COLLECT_SEVERITY,
+    profilerWindowLength: process.env.TRACE_PROFILER_WINDOW_LENGTH
+      ? parseInt(process.env.TRACE_PROFILER_WINDOW_LENGTH)
+      : 10000
   }
 
   if (process.env.TRACE_SAMPLER_LIMIT) {