Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Export zone locality in outbound destination metrics #13129

Merged
merged 1 commit into from
Oct 15, 2024
Merged

Conversation

sfleen
Copy link
Contributor

@sfleen sfleen commented Oct 2, 2024

Currently, we don't have a simple way of checking if the endpoint a proxy is discovering is in the same zone or not.

This adds a "zone_locality" metric label to the outbound destination address metrics. Note that this does not increase the cardinality of the related metrics, as this label doesn't vary within an endpoint.

Validated by checking the prometheus metrics on a local cluster and verifying this label appears in the outbound transport metrics.

@sfleen sfleen requested a review from a team as a code owner October 2, 2024 16:31
@sfleen sfleen force-pushed the zone branch 2 times, most recently from e908885 to d6580f4 Compare October 9, 2024 12:54
Copy link
Member

@adleong adleong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@sfleen sfleen force-pushed the zone branch 4 times, most recently from d9a4e1a to 1c3ce48 Compare October 15, 2024 18:03
Currently, we don't have a simple way of checking if the endpoint a proxy is discovering is in the same zone or not.

This adds a "zone_locality" metric label to the outbound destination address metrics. Note that this does not increase the cardinality of the related metrics, as this label doesn't vary within an endpoint.

Validated by checking the prometheus metrics on a local cluster and verifying this label appears in the outbound transport metrics.

Signed-off-by: Scott Fleener <[email protected]>
@adleong adleong merged commit 958cfca into linkerd:main Oct 15, 2024
38 checks passed
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