Skip to content

Conversation

cratelyn
Copy link
Member

@cratelyn cratelyn commented Oct 17, 2025

nb: this branch is based upon #4250.

  • feat(http/prom): add http and grpc status labelers
    this commit introduces a collection of MkStreamLabel and StreamLabel
    implementations that can be used to inspect status codes for HTTP and
    gRPC traffic.

    these do not emit duration labels, but can be used by other
    implementors of these traits to perform the common logic of checking
    HTTP status codes in response front matter, and for inspecting body
    trailers in gRPC traffic.

  • feat(http/prom): add an error labeler
    this commit introduces a LabelError<E> type, which provides a
    StreamLabel implementation that maps boxed errors to labels.

    this is generic across E-typed labels that can be constructed From a
    reference to a boxed linkerd_error::Error.

@cratelyn cratelyn self-assigned this Oct 17, 2025
this commit introduces a collection of `MkStreamLabel` and `StreamLabel`
implementations that can be used to inspect status codes for HTTP and
gRPC traffic.

these do not emit duration labels, but can be used by other
implementors of these traits to perform the common logic of checking
HTTP status codes in response front matter, and for inspecting body
trailers in gRPC traffic.

Signed-off-by: katelyn martin <[email protected]>
this commit introduces a `LabelError<E>` type, which provides a
`StreamLabel` implementation that maps boxed errors to labels.

this is generic across `E`-typed labels that can be constructed `From` a
reference to a boxed `linkerd_error::Error`.

Signed-off-by: katelyn martin <[email protected]>
@cratelyn cratelyn force-pushed the kate/http-prom.stream-label-components branch from aac9f88 to c8d28a6 Compare October 17, 2025 18:51
@cratelyn cratelyn changed the title kate/http prom.stream label components feat(http/prom): introduce StreamLabel components Oct 17, 2025
@cratelyn cratelyn marked this pull request as ready for review October 17, 2025 19:06
@cratelyn cratelyn requested a review from a team as a code owner October 17, 2025 19:06
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.

1 participant