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

[e2ealerting] Support pass through labels #207

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

callumj
Copy link

@callumj callumj commented Aug 9, 2021

We're using e2ealerting across our k8s fleet to monitor latency from ingestor to AM and want to be able to break that down by specific tags like cluster name or AZ. We could solve this by generating one rule for each but this becomes harder to manage as our cluster grows automatically. Instead this adds the ability to pass through custom labels from AM to the histogram and improves the timestamp seen logic to track this by the particular facets.

This adds 2 configuration parameters:

  • receiver.labels-to-track: This controls the label values of a metric that the "seen timestamp" will differentiate. This is useful if you want to track a different timestamp per host
  • receiver.labels-to-forward: This controls the labels that will be forwarded from the AM payload to the histogram metric. Useful to break the histogram down by cluster or region.

Allow for customizing the labels included in the final end_to_end_duration_seconds histogram, allowing multiple probs to be deployed and the latency split by labels forwarded to it from AlertManager.
@callumj callumj requested a review from a team as a code owner August 9, 2021 23:04
@callumj
Copy link
Author

callumj commented Feb 7, 2022

@gotjosh Mind reviewing this?

@CLAassistant
Copy link

CLAassistant commented Jun 15, 2022

CLA assistant check
All committers have signed the CLA.

friedrichg pushed a commit to cortexproject/cortex-tools that referenced this pull request Aug 1, 2023
Signed-off-by: Tom Hayward <thayward@infoblox.com>
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.

None yet

2 participants