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

[AO] Alert details page - Investigation about main chart of Metric Threshold rule type #151412

Closed
Tracked by #145117
maryam-saeidi opened this issue Feb 16, 2023 · 1 comment Β· Fixed by #152697
Closed
Tracked by #145117
Assignees
Labels
Feature:Alert Details Page Observability ux management team Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.8.0

Comments

@maryam-saeidi
Copy link
Member

maryam-saeidi commented Feb 16, 2023

πŸ“ Summary

Design: Figma

We need to check what needs to be done to use the Metric Explorer chart on the Alert details page.

image

Metric explorer chart:

image

βœ… Acceptance Criteria

  • Answer the following questions:

    • How easily can we use the Metric Explorer chart on the Alert details page? Is there any issue or blocker for it? (Maybe we need to fix this issue before using the chart)

    We decided to use preview charts instead of the metric explorer one since it has everything that we need for the alert details page. We only need to make some adjustments, such as allowing a custom time range, and using a line chart instead of a bar chart, ...

    • What is the road map for Metric Explorer page?

    • Can we use a Lens visualization for this purpose?

    It would be easier to use preview for the first sound since being able to use the charts in dashboards is not a hard requirement at this step. After implementing the alert details page for the metric threshold, we have a better overview of the complexity, and we can plan afterward to move to the Lens visualization if needed.

    • What type of chart do we want to use by default?

    In the first step, we use line charts, but we can have a dedicated ticket later on to have more specific defaults.

    • How should we handle the custom equation chart?

    Since we already have a preview for it, we can use the same on the alert details page.

@maryam-saeidi maryam-saeidi added Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.8.0 labels Feb 16, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/actionable-observability (Team: Actionable Observability)

@maryam-saeidi maryam-saeidi self-assigned this Feb 16, 2023
maryam-saeidi added a commit that referenced this issue Mar 21, 2023
…52697)

Closes #151412

## πŸ“ Summary 
This PR adds preview charts to the metric threshold's rule details page.

![image](https://user-images.githubusercontent.com/12370520/223083505-522dbb7c-d65f-4c79-bcd1-9cf04923d964.png)

**Note**
- This is only the first step, so there are limitations that will be
tackled in the follow-up PRs, such as:
  - Allow custom time range for preview chart
  - Adding extra alert annotation on the preview charts
  - Sync chart pointers

## βœ… Acceptance Criteria
- Create a metric threshold rule that fires and check the related alert
details page
@maryam-saeidi maryam-saeidi added the Feature:Alert Details Page Observability ux management team label Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Alert Details Page Observability ux management team Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.8.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants