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.

Sign up for GitHub

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

Jump to bottom

[Metrics UI] P95/P99 charts not rendering in Metrics Explorer and Alerts throws exeception #65561

Closed
simianhacker opened this issue May 6, 2020 · 1 comment · Fixed by #65579
Closed

[Metrics UI] P95/P99 charts not rendering in Metrics Explorer and Alerts throws exeception #65561

simianhacker opened this issue May 6, 2020 · 1 comment · Fixed by #65579
Assignees
@simianhacker
Labels
bug Fixes for quality problems that affect the customer experience 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 v7.8.0

Comments

@simianhacker
Copy link
Member

Kibana version:
master, 7.8
Elasticsearch version:
master, 7.8
Describe the bug:
When you render a chart with either 95ty Percentile or 99th Percentile the charts are empty. When you create an alert from the Metrics Explorer UI using percentiles the Alerts dialog throws an exception

Steps to reproduce:

  1. Open Metrics Explorer
  2. Set aggregation to 95th percentile
  3. Notice no data in the charts
  4. Click on "Actions" and "Create alert"
  5. BOOM!

Expected behavior:

It should render data and not throw the exception in alerts

Screenshots (if relevant):

Alerts Error

image
@simianhacker simianhacker added bug Fixes for quality problems that affect the customer experience [zube]: In Progress 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 v7.8.0 labels May 6, 2020
@simianhacker simianhacker self-assigned this May 6, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

simianhacker added a commit to simianhacker/kibana that referenced this issue May 6, 2020
@simianhacker
[Metrics UI] Fix p95/p99 charts and alerting error
a847cc5 
- Fixes elastic#65561
@simianhacker simianhacker mentioned this issue May 6, 2020
Merged
1 task
simianhacker added a commit that referenced this issue May 8, 2020
@simianhacker
[Metrics UI] Fix p95/p99 charts and alerting error (#65579)
1c6e6cb 
* [Metrics UI] Fix p95/p99 charts and alerting error

- Fixes #65561

* Fixing open in visualize for percentiles

* Adding test for P95; refactoring to use first consitently
simianhacker added a commit to simianhacker/kibana that referenced this issue May 8, 2020
@simianhacker
[Metrics UI] Fix p95/p99 charts and alerting error (elastic#65579)
021da34 
* [Metrics UI] Fix p95/p99 charts and alerting error

- Fixes elastic#65561

* Fixing open in visualize for percentiles

* Adding test for P95; refactoring to use first consitently
simianhacker added a commit to simianhacker/kibana that referenced this issue May 8, 2020
@simianhacker
[Metrics UI] Fix p95/p99 charts and alerting error (elastic#65579)
1099f21 
* [Metrics UI] Fix p95/p99 charts and alerting error

- Fixes elastic#65561

* Fixing open in visualize for percentiles

* Adding test for P95; refactoring to use first consitently
simianhacker added a commit that referenced this issue May 11, 2020
@simianhacker @elasticmachine
[Metrics UI] Fix p95/p99 charts and alerting error (#65579) (#65895)
0a1ec76 
* [Metrics UI] Fix p95/p99 charts and alerting error

- Fixes #65561

* Fixing open in visualize for percentiles

* Adding test for P95; refactoring to use first consitently

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
simianhacker added a commit that referenced this issue May 11, 2020
@simianhacker @elasticmachine
[Metrics UI] Fix p95/p99 charts and alerting error (#65579) (#65896)
c6b398d 
* [Metrics UI] Fix p95/p99 charts and alerting error

- Fixes #65561

* Fixing open in visualize for percentiles

* Adding test for P95; refactoring to use first consitently

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
@zube zube bot removed the [zube]: Done label Oct 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@simianhacker simianhacker

Labels
bug Fixes for quality problems that affect the customer experience 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 v7.8.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Metrics UI] Fix p95/p99 charts and alerting error simianhacker/kibana
2 participants
@simianhacker @elasticmachine