-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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] Add static mapping for metricsets in aws module #17650
Conversation
Pinging @elastic/integrations-platforms (Team:Platforms) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good. Just left some notes. Didn't test them, but maybe we need to so as to verify these fields' types, since some of them, like sums
, could be tricky, wdyt?
💔 Build FailedExpand to view the summary
Build stats
Test stats 🧪
Test errorsExpand to view the tests failures
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
CI failure is in
|
What does this PR do?
This PR adds static mappings for light weight metricsets in aws module: billing, ebs, elb, lambda, natgateway, sns, transitgateway, usage, vpn.
Why is it important?
To avoid having breaking non TSVB visualisations when there no data in ES to create the respective fields with the dynamic mapping.
This PR is an extension for #17614.