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

add indexdump rules analyzer#1840

Merged
Dieterbe merged 4 commits intomasterfrom
index-rules-analyzer
Dec 1, 2020
Merged

add indexdump rules analyzer#1840
Dieterbe merged 4 commits intomasterfrom
index-rules-analyzer

Conversation

@Dieterbe
Copy link
Copy Markdown
Contributor

@Dieterbe Dieterbe commented Jun 1, 2020

No description provided.

@Dieterbe Dieterbe force-pushed the index-rules-analyzer branch from d8328f2 to 988177a Compare June 1, 2020 20:11
@Dieterbe Dieterbe force-pushed the index-rules-analyzer branch from 988177a to 5c1179b Compare June 1, 2020 20:17
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown

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.

2 participants