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

Document default metricsets #12502

Closed
urso opened this issue Jun 11, 2019 · 9 comments
Closed

Document default metricsets #12502

urso opened this issue Jun 11, 2019 · 9 comments
Labels
docs enhancement Team:Integrations Label for the Integrations team

Comments

@urso
Copy link

urso commented Jun 11, 2019

Metricbeat modules start a set of default metricsets, if no metricset is configured, or if the list of metricsets is empty. The set of default metricsets is based on internal knowledge available in code only. Without reading the code it is quite difficult to learn the default metricsets to be run.

Request:

  • Document default metricsets
  • Make it clear in docs that an empty list will load the default ones.
  • (wishfull thinking) Have the info available in a modules _meta section and have support for automated testing.
@ruflin ruflin added the Team:Integrations Label for the Integrations team label Jun 11, 2019
@ruflin
Copy link
Contributor

ruflin commented Jun 11, 2019

Added the :infrastructure label as I think the integrations team should take care of this. I was thinking we already document this for most modules but need to double check.

@fearful-symmetry
Copy link
Contributor

Agreed, I think this is a good idea.

@fearful-symmetry
Copy link
Contributor

Okay actually looking into this now.
So, default metricsets are defined by mb.DefaultMetricSet() in init() Using that to generate the actual list of defaults isn't gonna be hard. Where do we want to document this? In modules.d/*.yml somehow? In the asciidocs for the modules? in _meta ?

(wishfull thinking) Have the info available in a modules _meta section and have support for automated testing.

@urso Automated testing of what? The correctness of the list of default modules?

@ruflin
Copy link
Contributor

ruflin commented Jun 13, 2019

Having it documented in the docs would be great.

Thinking ahead for lightweight modules it probably has to be specified in the module fields.yml (or best case, generated to keep it in sync). This we can then use for testing and generating docs.

@fearful-symmetry
Copy link
Contributor

Thinking ahead for lightweight modules it probably has to be specified in the module fields.yml

I wonder if we can have it in just the fields.yml as opposed to init() ?

Either way, I'll put together a PR to document it in the module-level docs.

@ruflin
Copy link
Contributor

ruflin commented Jun 13, 2019

Good question, agree that we don't necessarly need it in the init part. Would mean a change to the framework but sounds like a good change if we can do it.

@fearful-symmetry
Copy link
Contributor

Yah, init is kinda a weird place for it, and it took me forever to track down at first.

I'm also thinking that once we have it in the yml, we could easily generate a === Default Modules asciidoc blurb.

@dedemorton
Copy link
Contributor

@fearful-symmetry Can we close this issue now?

@fearful-symmetry
Copy link
Contributor

Yep!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs enhancement Team:Integrations Label for the Integrations team
Projects
None yet
Development

No branches or pull requests

5 participants