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

[Synthetics] Large numbers of monitors break graphs on the Overview page #187264

Closed
justinkambic opened this issue Jul 1, 2024 · 3 comments · Fixed by #199512
Closed

[Synthetics] Large numbers of monitors break graphs on the Overview page #187264

justinkambic opened this issue Jul 1, 2024 · 3 comments · Fixed by #199512
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Synthetics Team:obs-ux-management Observability Management User Experience Team

Comments

@justinkambic
Copy link
Contributor

Kibana version:

8.14

Elasticsearch version:

8.14

Original install method (e.g. download page, yum, from source, etc.):

Cloud

Describe the bug:

When hosting a large number of Synthetics monitors (it seems around 1K+), it is possible to break the charts at the top of the page. This is likely due to bucket overflow in the underlying ES queries, as when the user reduces the data via filtering the charts continue working.

Steps to reproduce:

  1. Create many monitors in cloud
  2. Attempt to view the charts on the overview page
  3. See the chart break

Expected behavior:

The chart should scale regardless of the number of monitors in use.

Screenshots (if relevant):

image

Errors in browser console (if relevant):

Provide logs and/or server output (if relevant):

Any additional context:

@justinkambic justinkambic added bug Fixes for quality problems that affect the customer experience Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Jul 1, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@smith smith added Synthetics Team:obs-ux-management Observability Management User Experience Team and removed Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Jul 2, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@jasonrhodes
Copy link
Member

Probably an ES error around "too many buckets" or similar?

@shahzad31 shahzad31 self-assigned this Oct 28, 2024
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Nov 21, 2024
…lastic#199512)

## Summary

Fixes elastic#187264 !!

Apply filters directly instead of passing each monitor id !!

### Testing

No special testing is needed, other than make sure, alerts/errors vizs
continue to work as expected !!

<img width="1726" alt="image"
src="https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503">

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 944e6fa)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Nov 21, 2024
…lastic#199512)

## Summary

Fixes elastic#187264 !!

Apply filters directly instead of passing each monitor id !!

### Testing

No special testing is needed, other than make sure, alerts/errors vizs
continue to work as expected !!

<img width="1726" alt="image"
src="https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503">

---------

Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 944e6fa)
kibanamachine added a commit that referenced this issue Nov 21, 2024
…rs !! (#199512) (#201080)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Synthetics] Fix overview page vizs for large number of monitors !!
(#199512)](#199512)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"Shahzad","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-21T07:27:41Z","message":"[Synthetics]
Fix overview page vizs for large number of monitors !! (#199512)\n\n##
Summary\r\n\r\nFixes #187264
!!\r\n\r\nApply filters directly instead of passing each monitor id
!!\r\n\r\n### Testing\r\n\r\nNo special testing is needed, other than
make sure, alerts/errors vizs\r\ncontinue to work as expected
!!\r\n\r\n<img width=\"1726\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503\">\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"944e6fa0376702342bb37c3c9893e1574689b211","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","ci:project-deploy-observability","Team:obs-ux-management","v8.16.0","backport:version","v8.17.0"],"title":"[Synthetics]
Fix overview page vizs for large number of monitors
!!","number":199512,"url":"https://github.com/elastic/kibana/pull/199512","mergeCommit":{"message":"[Synthetics]
Fix overview page vizs for large number of monitors !! (#199512)\n\n##
Summary\r\n\r\nFixes #187264
!!\r\n\r\nApply filters directly instead of passing each monitor id
!!\r\n\r\n### Testing\r\n\r\nNo special testing is needed, other than
make sure, alerts/errors vizs\r\ncontinue to work as expected
!!\r\n\r\n<img width=\"1726\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503\">\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"944e6fa0376702342bb37c3c9893e1574689b211"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.17"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199512","number":199512,"mergeCommit":{"message":"[Synthetics]
Fix overview page vizs for large number of monitors !! (#199512)\n\n##
Summary\r\n\r\nFixes #187264
!!\r\n\r\nApply filters directly instead of passing each monitor id
!!\r\n\r\n### Testing\r\n\r\nNo special testing is needed, other than
make sure, alerts/errors vizs\r\ncontinue to work as expected
!!\r\n\r\n<img width=\"1726\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503\">\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"944e6fa0376702342bb37c3c9893e1574689b211"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Shahzad <[email protected]>
mbondyra pushed a commit to mbondyra/kibana that referenced this issue Nov 21, 2024
…lastic#199512)

## Summary

Fixes elastic#187264 !!

Apply filters directly instead of passing each monitor id !!

### Testing

No special testing is needed, other than make sure, alerts/errors vizs
continue to work as expected !!

<img width="1726" alt="image"
src="https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503">

---------

Co-authored-by: kibanamachine <[email protected]>
kibanamachine added a commit that referenced this issue Nov 21, 2024
…rs !! (#199512) (#201081)

# Backport

This will backport the following commits from `main` to `8.17`:
- [[Synthetics] Fix overview page vizs for large number of monitors !!
(#199512)](#199512)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"Shahzad","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-21T07:27:41Z","message":"[Synthetics]
Fix overview page vizs for large number of monitors !! (#199512)\n\n##
Summary\r\n\r\nFixes #187264
!!\r\n\r\nApply filters directly instead of passing each monitor id
!!\r\n\r\n### Testing\r\n\r\nNo special testing is needed, other than
make sure, alerts/errors vizs\r\ncontinue to work as expected
!!\r\n\r\n<img width=\"1726\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503\">\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"944e6fa0376702342bb37c3c9893e1574689b211","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","ci:project-deploy-observability","Team:obs-ux-management","v8.16.0","backport:version","v8.17.0"],"title":"[Synthetics]
Fix overview page vizs for large number of monitors
!!","number":199512,"url":"https://github.com/elastic/kibana/pull/199512","mergeCommit":{"message":"[Synthetics]
Fix overview page vizs for large number of monitors !! (#199512)\n\n##
Summary\r\n\r\nFixes #187264
!!\r\n\r\nApply filters directly instead of passing each monitor id
!!\r\n\r\n### Testing\r\n\r\nNo special testing is needed, other than
make sure, alerts/errors vizs\r\ncontinue to work as expected
!!\r\n\r\n<img width=\"1726\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503\">\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"944e6fa0376702342bb37c3c9893e1574689b211"}},"sourceBranch":"main","suggestedTargetBranches":["8.16","8.17"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199512","number":199512,"mergeCommit":{"message":"[Synthetics]
Fix overview page vizs for large number of monitors !! (#199512)\n\n##
Summary\r\n\r\nFixes #187264
!!\r\n\r\nApply filters directly instead of passing each monitor id
!!\r\n\r\n### Testing\r\n\r\nNo special testing is needed, other than
make sure, alerts/errors vizs\r\ncontinue to work as expected
!!\r\n\r\n<img width=\"1726\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503\">\r\n\r\n---------\r\n\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"944e6fa0376702342bb37c3c9893e1574689b211"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Shahzad <[email protected]>
Co-authored-by: Brad White <[email protected]>
paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this issue Nov 26, 2024
…lastic#199512)

## Summary

Fixes elastic#187264 !!

Apply filters directly instead of passing each monitor id !!

### Testing

No special testing is needed, other than make sure, alerts/errors vizs
continue to work as expected !!

<img width="1726" alt="image"
src="https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503">

---------

Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Dec 12, 2024
…lastic#199512)

## Summary

Fixes elastic#187264 !!

Apply filters directly instead of passing each monitor id !!

### Testing

No special testing is needed, other than make sure, alerts/errors vizs
continue to work as expected !!

<img width="1726" alt="image"
src="https://github.com/user-attachments/assets/9c1889a5-4822-442b-97af-c2a4084c4503">

---------

Co-authored-by: kibanamachine <[email protected]>
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 Synthetics Team:obs-ux-management Observability Management User Experience Team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants