From d065905e73da66c35e1ed4c1ed580f4d798cd1f9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ivar=20Conradi=20=C3=98sthus?= Date: Tue, 26 Mar 2024 12:25:15 +0100 Subject: [PATCH] fix: Add metrics for old proxy forward (#6695) This change adds a new prometheus counter to all us to capture when we automatically forward traffic from old /proxy paths to the /api/frontend path. ![image](https://github.com/Unleash/unleash/assets/158948/639a4ade-4758-41e6-b87b-a497f00313fa) --- src/lib/metrics.ts | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/src/lib/metrics.ts b/src/lib/metrics.ts index 2643ba67c6..0f412f3acd 100644 --- a/src/lib/metrics.ts +++ b/src/lib/metrics.ts @@ -103,6 +103,11 @@ export default class MetricsMonitor { help: 'Number of times a feature toggle has been used', labelNames: ['toggle', 'active', 'appName'], }); + const oldEmbeddedProxyForwardCounter = createCounter({ + name: 'old_embedded_proxy_forward_total', + help: 'Number of times we forwarded old embedded proxy requests', + labelNames: ['path', 'method'], + }); const featureTogglesTotal = createGauge({ name: 'feature_toggles_total', help: 'Number of feature toggles', @@ -600,6 +605,11 @@ export default class MetricsMonitor { } }); + // On purpose a hard-coded string. Will we removed when we are done migrating. + eventBus.on('OLD_EMBEDDED_PROXY_REQUEST', ({ path, method }) => { + oldEmbeddedProxyForwardCounter.labels({ path, method }).inc(); + }); + await this.configureDbMetrics(db, eventBus, schedulerService); return Promise.resolve();