Skip to content

Conversation

@phillipb
Copy link
Contributor

Summary

The alert flyout doesn't make sense on the inventory page. That page will likely require a different alert type that we'll be introducing in 7.8.

Also, since none of our metrics report at the "seconds" level, default the flyout to minutes. In the future, we should add a check that queries the backend to find the minimum duration for the metrics in the alert condition.

@phillipb phillipb added Feature:Metrics UI Metrics UI feature v8.0.0 Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services release_note:skip Skip the PR/issue when compiling release notes v7.7.0 labels Mar 24, 2020
@phillipb phillipb requested a review from a team as a code owner March 24, 2020 12:42
@elasticmachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Metrics UI Metrics UI feature release_note:skip Skip the PR/issue when compiling release notes Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v7.7.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants