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

AWS Billing module EstimatedCharges field doesn't match integration #1592

Closed
legoguy1000 opened this issue Sep 8, 2021 · 9 comments · Fixed by #1606
Closed

AWS Billing module EstimatedCharges field doesn't match integration #1592

legoguy1000 opened this issue Sep 8, 2021 · 9 comments · Fixed by #1606
Assignees

Comments

@legoguy1000
Copy link
Contributor

legoguy1000 commented Sep 8, 2021

The Metricbeat AWS Billing module sets the field EstimatedCharges as a Long. The AWS Billing Integration data stream for the agent sets EstimatedCharges.max as a Long. User reported an error with the AWS agent integration https://elasticstack.slack.com/archives/CNTP6NLTZ/p1631079578142800?thread_ts=1631079307.142600&cid=CNTP6NLTZ.

(status=400): {"type":"mapper_parsing_exception","reason":"object mapping for [aws.billing.EstimatedCharges] tried to parse field [EstimatedCharges] as object, but found a concrete value"}

The change in the Metricbeat module from EstimatedCharges.max to EstimatedCharges was made in elastic/beats#20527
The AWS Integration needs to update its field mapping to match the Metricbeat module.

@legoguy1000
Copy link
Contributor Author

@kaiyan-sheng thoughts?

@kaiyan-sheng
Copy link
Contributor

Yes this is a bug caused by not syncing the change between beats and the integration 👍 Thanks for catching it! Will fix it right away.

@kaiyan-sheng kaiyan-sheng self-assigned this Sep 8, 2021
@legoguy1000
Copy link
Contributor Author

I have a branch already, just hadn't pushed it yet. I can push it shortly for u too look at if u want or if there are other changes u want to make as well, I can just toss it.

@kaiyan-sheng
Copy link
Contributor

@legoguy1000 😂 sorry!!! I just saw your comment...!!

@legoguy1000
Copy link
Contributor Author

@legoguy1000 😂 sorry!!! I just saw your comment...!!

no problem, i figured there were other changes that needed to be made too. Glad it was fixed.

@kaiyan-sheng
Copy link
Contributor

Thank you for creating the issue!!!

@threenub
Copy link

threenub commented Sep 8, 2021

Thanks all - as the user that reported it let me know if I can help or test in any way.

@threenub
Copy link

Hi @kaiyan-sheng -- how long roughly until this fix makes it into the repo?

@kaiyan-sheng
Copy link
Contributor

@threenub The PR is merged into 0.10.6 version of the AWS package, which is still in staging right now. It will be pushed into production in next day or two.

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 a pull request may close this issue.

3 participants