Parcourir la source

avoid step effects in summary metrics

reduce the window size from 10 minutes to 1 minute,  so that short
spikes do not cause a 10 minute long "table" graph.
Brian Gough il y a 6 ans
Parent
commit
03e81153db
2 fichiers modifiés avec 2 ajouts et 2 suppressions
  1. 1 1
      libraries/metrics/package.json
  2. 1 1
      libraries/metrics/prom_wrapper.coffee

+ 1 - 1
libraries/metrics/package.json

@@ -1,6 +1,6 @@
 {
 {
   "name": "metrics-sharelatex",
   "name": "metrics-sharelatex",
-  "version": "2.5.0",
+  "version": "2.5.1",
   "description": "A drop-in metrics and monitoring module for node.js apps",
   "description": "A drop-in metrics and monitoring module for node.js apps",
   "repository": {
   "repository": {
     "type": "git",
     "type": "git",

+ 1 - 1
libraries/metrics/prom_wrapper.coffee

@@ -55,7 +55,7 @@ class MetricWrapper
 				new prom.Summary({
 				new prom.Summary({
 					name: name,
 					name: name,
 					help: name,
 					help: name,
-					maxAgeSeconds: 600,
+					maxAgeSeconds: 60,
 					ageBuckets: 10,
 					ageBuckets: 10,
 					labelNames: ['app', 'host', 'path', 'status_code', 'method', 'collection', 'query']
 					labelNames: ['app', 'host', 'path', 'status_code', 'method', 'collection', 'query']
 				})
 				})