Skip to content

Commit e35f97a

Browse files
authored
outbound: Set logical address on endpoints (#454)
Our metrics expect the `authority` to be set to the logical address, not the concrete address (i.e. when TrafficSplit may override the concrete destination). The concrete service is annotated via endpoint labels.
1 parent 871e56c commit e35f97a

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

linkerd/app/outbound/src/endpoint.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,8 @@ impl MapEndpoint<Concrete<http::Settings>, Metadata> for FromMetadata {
271271
});
272272

273273
Target {
274-
addr: concrete.addr.clone(),
274+
// Use the logical addr for the target.
275+
addr: concrete.inner.addr.clone(),
275276
inner: HttpEndpoint {
276277
addr,
277278
identity,

0 commit comments

Comments
 (0)