fix: wait 60 seconds before measuring nodejs.cpuUsage so that you don…

…'t get an artificially high first reading (#35)
Netflix-Skunkworks · Nov 21, 2024 · e127443 · e127443
1 parent a6afd9c
commit e127443
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/index.js b/src/index.js
@@ -278,7 +278,6 @@ class RuntimeMetrics {
   }
 
   _cpuHeap() {
-    RuntimeMetrics.measureCpuHeap(this);
     const reg = this.registry;
     this._intervals.push(reg.schedulePeriodically(RuntimeMetrics.measureCpuHeap, 60000, this));
   }

diff --git a/test/nodemetrics.test.js b/test/nodemetrics.test.js
@@ -23,7 +23,7 @@ describe('nodemetrics', () => {
 
     setTimeout(() => {
       const meters = registry.meters();
-      assert.isTrue(meters.length > 40);
+      assert.isTrue(meters.length > 15);
       metrics.stop();
       done();
     });