Skip to content

outbound: Emit INFO-level logs on failure accrual changes#2373

Merged
olix0r merged 4 commits intomainfrom
ver/failure-accrual-logs
Apr 4, 2023
Merged

outbound: Emit INFO-level logs on failure accrual changes#2373
olix0r merged 4 commits intomainfrom
ver/failure-accrual-logs

Conversation

@olix0r
Copy link
Member

@olix0r olix0r commented Apr 1, 2023

There are no explicit indications of failure accrual related behavior.

This change adds INFO-level logs when a failure accrual breaker is
tripped or reeopened. These log messages are scoped within the
'endpoint' log context so that log lines include endpoint and balancer
addresses.

INFO balance{addr=logical.test.svc.cluster.local:666}:endpoint{addr=192.0.2.41:666}:consecutive_failures: linkerd_app_outbound::http::breaker::consecutive_failures: Consecutive failure-accrual breaker closed
INFO balance{addr=logical.test.svc.cluster.local:666}:endpoint{addr=192.0.2.41:666}:consecutive_failures: linkerd_app_outbound::http::breaker::consecutive_failures: Consecutive failure-accrual breaker reopened

olix0r added 3 commits April 1, 2023 21:22
There are no explicit indications of failure accrual related behavior.

This change adds INFO-level logs when a failure accrual breaker is
tripped or reeopened. These log messages are scoped within the
'endpoint' log context so that log lines include endpoint and balancer
addresses.

    INFO balance{addr=logical.test.svc.cluster.local:666}:endpoint{addr=192.0.2.41:666}:consecutive_failures: linkerd_app_outbound::http::breaker::consecutive_failures: Consecutive failure-accrual breaker tripped
    INFO balance{addr=logical.test.svc.cluster.local:666}:endpoint{addr=192.0.2.41:666}:consecutive_failures: linkerd_app_outbound::http::breaker::consecutive_failures: Consecutive failure-accrual breaker reopened
@olix0r olix0r marked this pull request as ready for review April 4, 2023 00:07
@olix0r olix0r requested a review from a team as a code owner April 4, 2023 00:07
@olix0r olix0r merged commit 846ce1b into main Apr 4, 2023
@olix0r olix0r deleted the ver/failure-accrual-logs branch April 4, 2023 00:07
hawkw added a commit to linkerd/linkerd2 that referenced this pull request Apr 4, 2023
This proxy release adds new `outbound_http_route_backend_requests_total`
and `outbound_grpc_route_backend_requests_total` metrics, which can be
used to track how routing rules and backend distributions apply to
requests. These metrics contain labels describing the route's parent
(i.e. a Service), the route resource being used, and the backend
resource being used by each request. In addition, this proxy release
adds new `INFO`-level logs for failure accrual, and updates logging
contexts with Kubernetes resource metadata.

---

* outbound: Emit INFO-level logs on failure accrual changes (linkerd/linkerd2-proxy#2373)
* app: Rename Metrics types as InboundMetrics and OutboundMetrics (linkerd/linkerd2-proxy#2376)
* outbound: Configure balancers with service metadata (linkerd/linkerd2-proxy#2374)
* outbound: Report per-route-backend request count metrics (linkerd/linkerd2-proxy#2372)
* outbound: Report per-route-backend request count metrics (linkerd/linkerd2-proxy#2377)
hawkw added a commit to linkerd/linkerd2 that referenced this pull request Apr 5, 2023
This proxy release adds new `outbound_http_route_backend_requests_total`
and `outbound_grpc_route_backend_requests_total` metrics, which can be
used to track how routing rules and backend distributions apply to
requests. These metrics contain labels describing the route's parent
(i.e. a Service), the route resource being used, and the backend
resource being used by each request. In addition, this proxy release
adds new `INFO`-level logs for failure accrual, and updates logging
contexts with Kubernetes resource metadata.

---

* outbound: Emit INFO-level logs on failure accrual changes (linkerd/linkerd2-proxy#2373)
* app: Rename Metrics types as InboundMetrics and OutboundMetrics (linkerd/linkerd2-proxy#2376)
* outbound: Configure balancers with service metadata (linkerd/linkerd2-proxy#2374)
* outbound: Report per-route-backend request count metrics (linkerd/linkerd2-proxy#2372)
* outbound: Report per-route-backend request count metrics (linkerd/linkerd2-proxy#2377)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants