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

[Observability RAC] Meta: Register all observability rules with RAC registry system #96569

Open
2 of 5 tasks
smith opened this issue Apr 8, 2021 · 2 comments
Open
2 of 5 tasks
Labels
Epic: Observability RAC Rule Type Registration epic Meta Team:Observability Team label for Observability Team (for things that are handled across all of observability) Theme: rac label obsolete v7.15.0

Comments

@smith
Copy link
Contributor

smith commented Apr 8, 2021

To satisfy this requirement, each observability app should do the following:

  • Register their rule types with the RAC registry system
    • create rule registry that inherits from another rule registry (optional)
    • Add a reason handler
    • Add a url string to the app in the context of the alert (incl. time range, filters, etc.)
    • Specify a landing page that a user is taken to when clicking "View in app" on alerts table

Solutions:

This is related to #95908, which says,

First version will be an "hardcoded" reason defined in the product, users cannot customize it.

These can either be defined at when the rule is registered, or in the table renderers in the observability plugin. The table and flyouts should display and use the text and links.

@smith smith added Team:APM All issues that need APM UI Team support Theme: rac label obsolete labels Apr 8, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

@smith smith added Team:Observability Team label for Observability Team (for things that are handled across all of observability) v7.13.0 and removed Team:APM All issues that need APM UI Team support labels Apr 8, 2021
@smith smith closed this as completed Apr 8, 2021
@smith smith reopened this Apr 8, 2021
@smith smith added v7.14.0 and removed v7.13.0 labels Apr 12, 2021
@sorenlouv sorenlouv changed the title Reason renderers for observability apps [Observability] Reason renderers for observability apps Apr 13, 2021
@sorenlouv sorenlouv changed the title [Observability] Reason renderers for observability apps [Meta] Observability: migration path to from alert registry to rule registry Apr 13, 2021
@sorenlouv sorenlouv reopened this Apr 13, 2021
@sorenlouv sorenlouv changed the title [Meta] Observability: migration path to from alert registry to rule registry [Meta] Observability: migration path from "alert registry" to "rule registry" Apr 13, 2021
@sorenlouv sorenlouv changed the title [Meta] Observability: migration path from "alert registry" to "rule registry" [Observability RAC] Meta: migration path from "alert registry" to "rule registry" Apr 14, 2021
@jasonrhodes jasonrhodes changed the title [Observability RAC] Meta: migration path from "alert registry" to "rule registry" [Observability RAC] Meta: Register all observability rules with RAC registry system Apr 26, 2021
@paulb-elastic
Copy link
Contributor

Uptime issue for this: #98382

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic: Observability RAC Rule Type Registration epic Meta Team:Observability Team label for Observability Team (for things that are handled across all of observability) Theme: rac label obsolete v7.15.0
Projects
None yet
Development

No branches or pull requests

6 participants