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 Monitoring UI] Use "fit" function in Stack Monitoring graphs to communicate gaps in data #133135

Open
miltonhultgren opened this issue May 30, 2022 · 7 comments
Labels
enhancement New value added to drive a business result Feature:Stack Monitoring Team:Monitoring Stack Monitoring team

Comments

@miltonhultgren
Copy link
Contributor

If there is data missing due to connection issues or cluster overload, we get graphs with gaps like this:

We could make use of the fit function of the charts library to generate graphs like this instead:

@miltonhultgren miltonhultgren added Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services Feature:Stack Monitoring labels May 30, 2022
@elasticmachine
Copy link
Contributor

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

@markov00
Copy link
Member

@miltonhultgren @formgeist we, the datavis team, are reconsidering our thoughts on gap handling in charts, so I think it is a good issue to discuss about that topic.
The topic says communicate gaps in data and I like to ask you few questions about that:

  • how important are gaps with respect to the represented trend?
  • what are the different causes that, in this specific use case, can cause a gap to appear? are all at the same level?
  • I don't have enough knowledge about the topic, but looking at the chart we are talking about the indexing rate. If we have a gap, caused by a cluster overload, can we consider that indexing rate at zero? or it depends by other factors and, for example, the indexing rate is just unknown because the agent (or what is monitoring that rate) is unresponsive but we are still indexing the data?
  • do we want to be really precise and show exactly where those gaps are or do we want just a global warning about that fact?
  • ES has gaps policy, can be applied in your case?

@miltonhultgren
Copy link
Contributor Author

I'll defer most questions to our UX experts but in most cases I would say that the gap happens because the agent is having issues (either with collection or delivery), so the indexing rate is definitely unknown (not 0). So the gap tends to highlight an issue with the agent, and not an issue with the monitored system, so a global warning within the chart might be more accurate.
I'm not aware of what ES gap policies are.

@markov00
Copy link
Member

@miltonhultgren the current gap policies are described here https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations-pipeline.html#gap-policy
Basically you can mitigate those gaps by using zero, skipping the bucket or use a static value instead.
If the case is that the indexing rate is unknown then we should definitely understand if what is the importance of that information in relation to the overall monitoring trend, to avoid focusing on problems that are not primary for your user.

@miltonhultgren
Copy link
Contributor Author

Since we mostly care about communicating unknown values in this use case, it feels like gap policies won't help since reporting 0 is wrong and skipping also feels wrong as it would hide the gap?

@pmeresanu85 pmeresanu85 added the enhancement New value added to drive a business result label Sep 8, 2022
@pmeresanu85
Copy link

Added the enhancement label on this one

@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
enhancement New value added to drive a business result Feature:Stack Monitoring Team:Monitoring Stack Monitoring team
Projects
None yet
Development

No branches or pull requests

5 participants