Skip to content

Commit 8c6bbbb

Browse files
authored
Restore the route_actual_response_total metric (#468)
339efa8 incorrectly removed the route_actual metrics scope from the outbound stack. This change restores these metrics but omits latency buckets from export. Fixes linkerd/linkerd2#4215
1 parent fd3a4ac commit 8c6bbbb

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

linkerd/app/outbound/src/lib.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -263,6 +263,7 @@ impl Config {
263263

264264
let http_profile_route_proxy = svc::proxies()
265265
.check_new_clone_service::<dst::Route>()
266+
.push(metrics.http_route_actual.into_layer::<classify::Response>())
266267
// Sets an optional retry policy.
267268
.push(retry::layer(metrics.http_route_retry))
268269
.check_new_clone_service::<dst::Route>()

linkerd/app/src/metrics.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ impl Metrics {
4848
.clone()
4949
.into_report(retain_idle)
5050
.with_prefix("route_actual");
51-
(m, r)
51+
(m, r.without_latencies())
5252
};
5353

5454
let http_errors = errors::Metrics::default();

0 commit comments

Comments
 (0)