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

Update README.md #304

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

Conversation

jjuris
Copy link

@jjuris jjuris commented Mar 18, 2021

Hello

I would like to propose very simple change in documentation. I am missing a documentation of all metrics in this exporter.

Long story short

There are two teams in our company, monitoring and elastic team. Monitoring team asked me to provide all useful metrics I need to work with before they configure federation from elastic's team prometheus. The reason was that they do not want to federate all metrics, just a subset of useful ones. It was difficult for me to get all metrics because I had no access to the API of ElasticSearch. So I had to communicate with another team and ask them to provide me requested information.

TL;DR

I found it useful to have all metrics documented.

What do you think?

Best regards

Jozef

added metrics description table
@vvanholl vvanholl force-pushed the master branch 12 times, most recently from 9c3829c to daa4dbc Compare May 29, 2021 15:13
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