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

[Infra UI] Make interval in Metrics Explorer configurable #41907

Closed
kaiyan-sheng opened this issue Jul 24, 2019 · 9 comments
Closed

[Infra UI] Make interval in Metrics Explorer configurable #41907

kaiyan-sheng opened this issue Jul 24, 2019 · 9 comments
Assignees
Labels
Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@kaiyan-sheng
Copy link
Contributor

When testing Metricbeat rds metricset using Metrics Explorer, we realized instead of showing lines in the graph, it's showing data as individual dots:
Screen Shot 2019-07-24 at 16 31 00

But if displaying data in longer time range, then it shows lines instead of dots:
Screen Shot 2019-07-24 at 16 30 38

For rds metricset, metrics are reported every 1 minute. We suspect the bucket size Metrics Explorer uses is smaller than 1min. So when querying data, it gets empty data points and that causes the graph to be with individual dots.

@kaiyan-sheng
Copy link
Contributor Author

Possible solution is to add period into metrics that reported by Metricbeat so this value can be used by the interval for Metrics Explorer: elastic/beats#12616

@kaiyan-sheng kaiyan-sheng added the Feature:Metrics UI Metrics UI feature label Jul 24, 2019
@kaiyan-sheng
Copy link
Contributor Author

cc: @sorantis @exekias and @simianhacker

@sorantis sorantis added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Jul 24, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-logs-ui

@exekias
Copy link
Contributor

exekias commented Jul 25, 2019

I wonder if this could be fixed by only considering it a gap when time difference is > 5 mins? We get doc_count for each bucket on the responses, so differentiating empty buckets from 0 and discarding them should be doable.

@simianhacker
Copy link
Member

Since we agreed to tie Metrics Explorer to ECS we should use period if and when it becomes available. @Dimitri-Mazmanov From a product perspective, do we want to give the user a way to override the interval in the new chart configure UI?

@elastic elastic deleted a comment from zube bot Jul 30, 2019
@sorantis
Copy link

ignoring zube
@simianhacker, I think we shouldn't make the interval configurable in Metrics Explorer so long as we correctly show a graph using the period value exposed by Metricbeat. The users already have this option in TSVB if they choose to further customize the graph.

@simianhacker
Copy link
Member

I'm going to close this issue, if we feel like it's something we want to consider in the future then we can re-open it.

@roncohen
Copy link
Contributor

is there a way to just force it to connect the dots until period arrives?

@simianhacker
Copy link
Member

@roncohen If we did this, I would add it to the chart "customize" menu as "Ignore gaps & connect points" (or something that conveys the same message) since that's really what we will be doing. The caveat to "connecting the dots" is that it won't work for any of the rates unless there are at least 2 points next to each other in the raw data. Those two points will be required to calculate the derivative, the points from the derivative will be connected. Also keep in mind when we link to the TSVB charts, they will NOT connect the dots since there ins't away to tell TSVB to the ignore gaps.

@sorantis @roncohen If we decide to move forward with this, we should create a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

No branches or pull requests

7 participants