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

[Uptime] Fix broken overview page when no summary data present #81952

Merged
merged 1 commit into from
Oct 29, 2020

Conversation

andrewvc
Copy link
Contributor

@andrewvc andrewvc commented Oct 28, 2020

Fixes #81950 by not assuming the summary is present in a bucket with partial check
info

Summary

Summarize your PR. If it involves visual changes include a screenshot or gif.

Checklist

Delete any items that are not applicable to this PR.

For maintainers

Fixes elastic#81950

by not assuming the summary is present in a bucket with partial check
info
@andrewvc andrewvc requested a review from a team as a code owner October 28, 2020 20:08
@botelastic botelastic bot added the Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability label Oct 28, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@andrewvc andrewvc self-assigned this Oct 28, 2020
@andrewvc andrewvc added bug Fixes for quality problems that affect the customer experience v7.10.0 release_note:skip Skip the PR/issue when compiling release notes labels Oct 28, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@andrewvc andrewvc merged commit 6bff52c into elastic:master Oct 29, 2020
andrewvc added a commit to andrewvc/kibana that referenced this pull request Oct 29, 2020
…ic#81952)

Fixes elastic#81950

by not assuming the summary is present in a bucket with partial check
info
andrewvc added a commit to andrewvc/kibana that referenced this pull request Oct 29, 2020
…ic#81952)

Fixes elastic#81950

by not assuming the summary is present in a bucket with partial check
info
@andrewvc andrewvc deleted the handle-partial-checks-in-buckets branch October 29, 2020 15:44
andrewvc added a commit that referenced this pull request Oct 29, 2020
… (#82022)

Fixes #81950

by not assuming the summary is present in a bucket with partial check
info
andrewvc added a commit that referenced this pull request Oct 29, 2020
… (#82021)

Fixes #81950

by not assuming the summary is present in a bucket with partial check
info
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 release_note:skip Skip the PR/issue when compiling release notes Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Uptime] Synthetics data / missing summary data can break overview query
4 participants