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

[Stack Monitoring] Allow sub minute interval for CPU usage rule #136818

Open
klacabane opened this issue Jul 21, 2022 · 2 comments
Open

[Stack Monitoring] Allow sub minute interval for CPU usage rule #136818

klacabane opened this issue Jul 21, 2022 · 2 comments
Labels
bug Fixes for quality problems that affect the customer experience Feature:Stack Monitoring Team:Monitoring Stack Monitoring team v8.4.0

Comments

@klacabane
Copy link
Contributor

Summary

The cpu usage rule only supports minute as an interval unit and will convert any specified period to a minute. When dealing with a sub-minute interval this leads to a fraction and will break the execution with the following error:

x_content_parse_exception: [x_content_parse_exception] Reason: [1:852] [date_histogram] failed to parse field [fixed_interval], caused by: "failed to parse [0.9833333333333333m], fractional time values are not supported,For input string: "0.9833333333333333""

According to this comment it was a conscious decision but broke the seconds use case as a side effect.

We should investigate why converting the interval in ms with the date histogram aggregation doesn't work with this query and either fix the underlying issue or find an alternative that allows a sub-minute interval to be set.

AC

  • The cpu usage rule works with intervals inferior to a minute
@klacabane klacabane added bug Fixes for quality problems that affect the customer experience Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services Feature:Stack Monitoring v8.4.0 labels Jul 21, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@miltonhultgren
Copy link
Contributor

miltonhultgren commented Jul 21, 2022

The error itself is just normal stuff from the date_histogram aggs, fractional units of any size are not supported.

Fractional time values are not supported, but you can address this by shifting to another time unit (e.g., 1.5h could instead be specified as 90m).

That comment is very vague but I wonder if it's due to rounding issues?
Beyond that I can't think of any reason why we couldn't translate 0.9m into 54s.

@smith smith added Team:Monitoring Stack Monitoring team and removed Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience Feature:Stack Monitoring Team:Monitoring Stack Monitoring team v8.4.0
Projects
None yet
Development

No branches or pull requests

4 participants