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

Add AWS s3_daily_storage metricset #10940

Merged
merged 14 commits into from
Mar 1, 2019
Merged

Add AWS s3_daily_storage metricset #10940

merged 14 commits into from
Mar 1, 2019

Conversation

kaiyan-sheng
Copy link
Contributor

@kaiyan-sheng kaiyan-sheng commented Feb 25, 2019

This PR is to add s3_daily_storage metricset to AWS module.
Metrics:
bucket.name
bucket.size.bytes
number_of_objects

These metrics came from Amazon S3 CloudWatch Daily Storage Metrics for Buckets. These storage metrics for Amazon S3 are reported once per day, which indicates the collection period for s3_daily_storage metricset should be 86400s or multiple of 86400s.

#10055

@kaiyan-sheng kaiyan-sheng requested a review from a team as a code owner February 25, 2019 23:01
@kaiyan-sheng kaiyan-sheng self-assigned this Feb 25, 2019
@kaiyan-sheng kaiyan-sheng added Metricbeat Metricbeat Team:Integrations Label for the Integrations team labels Feb 25, 2019
@kaiyan-sheng kaiyan-sheng requested a review from a team as a code owner February 26, 2019 01:50
Copy link
Contributor

@sayden sayden left a comment

Choose a reason for hiding this comment

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

Overall looks good. Left some comments

@kaiyan-sheng
Copy link
Contributor Author

jenkins, test this please

kaiyan-sheng referenced this pull request Mar 1, 2019
* Add s3_request metricset for AWS module
* Add changelog
* Change metric names and add a check for timestamp
* Add ms unit to latency metrics
@kaiyan-sheng kaiyan-sheng merged commit 0acee7c into elastic:master Mar 1, 2019
@kaiyan-sheng kaiyan-sheng deleted the s3_daily branch March 1, 2019 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Metricbeat Metricbeat Team:Integrations Label for the Integrations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants