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

add indexdump rules analyzer #1840

Merged
merged 4 commits into from
Dec 1, 2020
Merged

add indexdump rules analyzer #1840

merged 4 commits into from
Dec 1, 2020

Conversation

Dieterbe
Copy link
Contributor

@Dieterbe Dieterbe commented Jun 1, 2020

No description provided.

Copy link
Contributor

@replay replay left a comment

Choose a reason for hiding this comment

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

Nice idea. So this reads the index-rules.conf and expects an index dump to be piped in the stdin, right? Might make sense to mention somewhere what format of the dump it expects. Otherwise, looks good.

@stale
Copy link

stale bot commented Aug 31, 2020

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 Aug 31, 2020
@Dieterbe
Copy link
Contributor Author

Might make sense to mention somewhere what format of the dump it expects

as the doc and the help text say:

reads metric names from stdin

@stale stale bot removed the stale label Aug 31, 2020
@stale
Copy link

stale bot commented Nov 30, 2020

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 30, 2020
@Dieterbe Dieterbe changed the title WIP / add indexdump rules analyzer add indexdump rules analyzer Dec 1, 2020
@stale stale bot removed the stale label Dec 1, 2020
@Dieterbe Dieterbe merged commit 5547d53 into master Dec 1, 2020
@Dieterbe Dieterbe deleted the index-rules-analyzer branch December 1, 2020 12:59
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.

None yet

2 participants