Improve resource.Quantity to float conversion for SLOs #1308
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
CONS-6488
Quantity.AsApproximateFloat64
does some math to convert Quantity tofloat64
needed for SLO API input so some floats may appear with additional decimals. Instead we convert Quantity to Decimal string and then parse as float. This may still have edge cases were string representation doesn't have exactfloat64
so conversion isn't 1-to-1.This impl ignores string to float parsing errors, assuming decimal string obtain from Quantity would be valid.
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Minimum Agent Versions
Are there minimum versions of the Datadog Agent and/or Cluster Agent required?
Describe your test plan
datadogSLOEnabled
Tested on user-reported combinations
Checklist
bug
,enhancement
,refactoring
,documentation
,tooling
, and/ordependencies
qa/skip-qa
label