Skip to content
This repository has been archived by the owner on Aug 23, 2023. It is now read-only.

Add tool for reporting out of order and duplicate metrics #2014

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

pub-djedruszczak
Copy link
Contributor

This PR adds a new tool, cmd/mt-kafka-mdm-report-out-of-order, which consumes metrics from Kafka and discovers those which are out of order or duplicates. It then groups these metrics by name or a specific tag using an index built from Cassandra, and outputs the results.

pub-djedruszczak and others added 7 commits December 1, 2021 12:29
… or tag (#112)

Adds mt-kafka-mdm-report-out-of-order, which consumes metrics from kafka and discovers those which are out of order or duplicates. It then groups these metrics by name or a specific tag using an index built from Cassandra, and outputs the results.
@pub-djedruszczak pub-djedruszczak force-pushed the feature-report-out-of-order branch from e4c50d8 to c16f94f Compare December 1, 2021 17:31
docs/tools.md Outdated Show resolved Hide resolved
Copy link
Contributor

@Dieterbe Dieterbe left a comment

Choose a reason for hiding this comment

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

this looks nice and more useful than the mt-kafka-mdm-sniff-out-of-order tool that we currently have.
i made 2 small suggestions, let's get those out of the way and then this can be merged.

I presume you've been running this successfully?

@stale
Copy link

stale bot commented May 3, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label May 3, 2022
Co-authored-by: Dieter Plaetinck <dieter@plaetinck.be>
@stale stale bot removed the stale label May 9, 2022
pub-djedruszczak and others added 2 commits May 9, 2022 14:19
Co-authored-by: Dieter Plaetinck <dieter@plaetinck.be>
@pub-djedruszczak pub-djedruszczak requested a review from Dieterbe May 9, 2022 18:33
@Dieterbe
Copy link
Contributor

I am going on a 3-4 month sabbatical starting now, and i think i was the last grafana maintainer.

@robert-milan
Copy link
Contributor

I'll be taking over while Dieter is out

@CLAassistant
Copy link

CLAassistant commented Jun 15, 2022

CLA assistant check
All committers have signed the CLA.

@stale
Copy link

stale bot commented Nov 2, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Nov 2, 2022
@shanson7 shanson7 removed the stale label Nov 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants