Skip to content

Commit 9e8c1fc

Browse files
committed
Merge branch 'main' into ver/route-reqs-mtx
2 parents df78f29 + 846ce1b commit 9e8c1fc

File tree

2 files changed

+14
-9
lines changed

2 files changed

+14
-9
lines changed

linkerd/app/outbound/src/http/breaker/consecutive_failures.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,12 @@ impl ConsecutiveFailures {
3333
return;
3434
}
3535

36+
tracing::info!("Consecutive failure-accrual breaker closed");
3637
if self.closed().await.is_err() {
3738
return;
3839
}
40+
41+
tracing::info!("Consecutive failure-accrual breaker reopened");
3942
}
4043
}
4144

linkerd/app/outbound/src/http/concrete.rs

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -120,15 +120,6 @@ impl<N> Outbound<N> {
120120
.instrument(|e: &Endpoint<T>| info_span!("forward", addr = %e.addr));
121121

122122
let endpoint = inner
123-
.push_on_service(
124-
rt.metrics
125-
.proxy
126-
.stack
127-
.layer(stack_labels("http", "endpoint")),
128-
)
129-
.instrument(|e: &Endpoint<T>| info_span!("endpoint", addr = %e.addr));
130-
131-
let balance = endpoint
132123
.push_map_target({
133124
let inbound_ips = inbound_ips.clone();
134125
move |((addr, metadata), target): ((SocketAddr, Metadata), Balance<T>)| {
@@ -164,6 +155,17 @@ impl<N> Outbound<N> {
164155
}
165156
}),
166157
)
158+
.push_on_service(svc::OnServiceLayer::new(
159+
rt.metrics
160+
.proxy
161+
.stack
162+
.layer(stack_labels("http", "endpoint")),
163+
))
164+
.push_on_service(svc::NewInstrumentLayer::new(
165+
|(addr, _): &(SocketAddr, _)| info_span!("endpoint", %addr),
166+
));
167+
168+
let balance = endpoint
167169
.push(http::NewBalancePeakEwma::layer(resolve))
168170
.push(svc::NewMapErr::layer_from_target::<ConcreteError, _>())
169171
.push_on_service(http::BoxResponse::layer())

0 commit comments

Comments
 (0)