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] Alert flyout is broken #73527

Closed
justinkambic opened this issue Jul 28, 2020 · 1 comment · Fixed by #73528
Closed

[Uptime] Alert flyout is broken #73527

justinkambic opened this issue Jul 28, 2020 · 1 comment · Fixed by #73528
Assignees
Labels
bug Fixes for quality problems that affect the customer experience Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan test-plan-ok issue has passed test plan v7.10.0 v8.0.0

Comments

@justinkambic
Copy link
Contributor

Kibana version:
master

Elasticsearch version:
master

Server OS version:
macOS

Browser version:
Chrome 83.x

Browser OS version:
macOS

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

Describe the bug:
The monitor status alert flyout is crashing with an error when it should display without issues.

Steps to reproduce:

  1. Open the Uptime UI
  2. Click the alerts dropdown and select Monitor Status
  3. Observe the error

Expected behavior:
The flyout should display controls for editing the alert.

Screenshots (if relevant):
Jul-28-2020 14-01-45

Errors in browser console (if relevant):

Uncaught TypeError: can't access property "errors", alertType.validate(...) is undefined

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

Any additional context:
This likely resulted from a recent optimization we added in #70632, which I approved. During the review I failed to notice the alert flyout wasn't working. Once we reenable the test in #72963 we should be resilient to failures like this.

@justinkambic justinkambic added bug Fixes for quality problems that affect the customer experience v8.0.0 Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability v7.10.0 v7.9.0 labels Jul 28, 2020
@justinkambic justinkambic self-assigned this Jul 28, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@justinkambic justinkambic removed their assignment Oct 7, 2020
@andrewvc andrewvc added the test-plan-ok issue has passed test plan label Oct 14, 2020
@andrewvc andrewvc self-assigned this Oct 14, 2020
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 Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability test-plan test-plan-ok issue has passed test plan v7.10.0 v8.0.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants