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] Add Observability Shared app to house shared components #154735

Closed
CoenWarmer opened this issue Apr 11, 2023 · 1 comment · Fixed by #154716
Closed

[Observability] Add Observability Shared app to house shared components #154735

CoenWarmer opened this issue Apr 11, 2023 · 1 comment · Fixed by #154716
Assignees
Labels
Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" v8.8.0

Comments

@CoenWarmer
Copy link
Contributor

CoenWarmer commented Apr 11, 2023

Part of #152783

📝 Summary

The problem
The UX app uses an Embeddable via the Exploratory View app on the Observability Overview page.
Therefore the Observability App requires the Exploratory View app.

The Exploratory View app needs to use the same Page Template as the Observability app in order to keep the same navigation structure as the rest of the Observability apps.

This relationship creates a cyclical dependency between these two apps. No bueno!

Potential solutions
There are three ways to solve this problem (perhaps more?):

  1. Have Synthetics team create a Overview page widget which does not use the Exploratory View app, so the Observability app does not need to depend on the Exploratory View app;
  2. Consolidate all Observability app inside one package so there are no ways for cyclical dependencies to occur;
  3. Create a separate app that houses components which are shared by all Observability apps, which does not depend on any of the Observability apps, so all downstream apps can safely depend on it.

✅ Acceptance criteria

  • There should be no cyclical dependencies between the Observability app and the Exploratory View app
  • The UX team should not have to recreate the Observability Overview widget
  • All Observability apps should be able to use the same Observability Page Template
@botelastic botelastic bot added the needs-team Issues missing a team label label Apr 11, 2023
@CoenWarmer CoenWarmer self-assigned this Apr 11, 2023
@CoenWarmer CoenWarmer added the Team: Actionable Observability - DEPRECATED For Observability Alerting and SLOs use "Team:obs-ux-management", for AIops "Team:obs-knowledge" label Apr 11, 2023
@elasticmachine
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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