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

[Metricbeat] Move statsd metricbeat module to GA #16447

Merged
merged 8 commits into from
Feb 20, 2020

Conversation

mtojek
Copy link
Contributor

@mtojek mtojek commented Feb 20, 2020

This PR prepares statsd module for releasing GA.

What has been done:

  • docs update
  • tags changed from beta to ga
  • system test

Issue: #14280

@mtojek mtojek added the Metricbeat Metricbeat label Feb 20, 2020
@mtojek mtojek requested a review from a team February 20, 2020 09:17
@mtojek mtojek self-assigned this Feb 20, 2020
The metricset collects metric data sent using UDP and publishes them under the `statsd` prefix.

[float]
=== Development
Copy link
Member

Choose a reason for hiding this comment

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

I think development notes should be in a README.md instead.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@jsoriano
Copy link
Member

Failure in docs build could be related, but not sure.

@mtojek
Copy link
Contributor Author

mtojek commented Feb 20, 2020

Failure in docs build could be related, but not sure.

Yes, I've been working on that in the mean time. Let the CI verify if it's fine now.

Copy link
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

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

Thanks!

@mtojek
Copy link
Contributor Author

mtojek commented Feb 20, 2020

Thanks!

Okey, it needed some time to propagate the visualization:

http://beats_16447.docs-preview.app.elstc.co/guide/en/beats/metricbeat/master/metricbeat-module-statsd.html

@mtojek mtojek merged commit c6701dd into elastic:master Feb 20, 2020
mtojek added a commit to mtojek/beats that referenced this pull request Feb 20, 2020
* Update docs
* Add system test
* Update changelog

(cherry picked from commit c6701dd)
@mtojek mtojek added the v7.7.0 label Feb 20, 2020
kvch pushed a commit to kvch/beats that referenced this pull request Feb 20, 2020
* Update docs
* Add system test
* Update changelog
mtojek added a commit that referenced this pull request Feb 21, 2020
* Update docs
* Add system test
* Update changelog

(cherry picked from commit c6701dd)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants