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

[Infra UI] Support using Data views in UI #125498

Closed
miltonhultgren opened this issue Feb 14, 2022 · 19 comments
Closed

[Infra UI] Support using Data views in UI #125498

miltonhultgren opened this issue Feb 14, 2022 · 19 comments
Labels
enhancement New value added to drive a business result Feature:Metrics UI Metrics UI feature needs-refinement A reason and acceptance criteria need to be defined for this issue Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team

Comments

@miltonhultgren
Copy link
Contributor

miltonhultgren commented Feb 14, 2022

Similar to the Logs UI, we should support Data views in the Infra UI

@miltonhultgren miltonhultgren added Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services needs-refinement A reason and acceptance criteria need to be defined for this issue labels Feb 14, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI)

@roshan-elastic
Copy link

Hey @miltonhultgren - given the split between observability and search + security, do you think Data Views will still be a thing for us?

@miltonhultgren
Copy link
Contributor Author

Deferring to @neptunian

@neptunian
Copy link
Contributor

Hey @miltonhultgren - given the split between observability and search + security, do you think Data Views will still be a thing for us?

@roshan-elastic I updated the description with a few use cases for supporting Data Views

@roshan-elastic
Copy link

Thanks @neptunian - I'd love to do this but will likely hold off until the direction of Infra UI is becoming clearer (esp w relation to server-less).

I'm not sure whether switching to Data Views something which be required long-term (not saying it isn't - I just don't know given serverless)...if it is then I'd really like to get this done but for now I don't have enough clarity on how serverless will impact the UI work we do

@roshan-elastic
Copy link

Hey @neptunian @jasonrhodes - do you think this will be important once we move to be asset-mgt powered?

Checking to see if this investment will be redundant once we move to that world...maybe we don't know yet...

@neptunian
Copy link
Contributor

I don't see how being asset-mgt powered affects the use cases in the description.

@roshan-elastic
Copy link

roshan-elastic commented Jun 26, 2023

Cool @neptunian - just checking in case being asset-powered would change anything.

I get that this would be nice to do and be a good user experience but I'm trying to get a sense of the value of this to understand how this should be prioritised, e.g. what would the user be able to do tomorrow if we were to release this today?

Is it best for me to speak with @miltonhultgren about this or maybe we should chat about this?

@neptunian
Copy link
Contributor

what would the user be able to do tomorrow if we were to release this today

There are various use cases for what being able to load custom Data Views into the UI. I tried to capture some in the description of this issue.

I don't think it's highest priority or that we should focus on it now.

@roshan-elastic
Copy link

Cool - sounds good @neptunian. Put this at the bottom of the backlog until it becomes worthwhile for us to focus on this.

@smith smith added Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team and removed Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Nov 14, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@neptunian
Copy link
Contributor

neptunian commented Mar 20, 2024

@roshan-elastic @smith I added #179053 to the description. Looks like this is a problem where users are trying view the asset being alerted on in our UIs using the data view they selected in the alert but will get errors due to us not supporting loading our UI with a provided data view.

@jasonrhodes
Copy link
Member

jasonrhodes commented Mar 20, 2024

For extra clarity, I'll just add that to solve the linked problems coming from alerts, we don't necessarily need the ability to specify a data view in the Infra UI settings screen (the way you can in Logs). (In fact, that likely wouldn't solve this problem, unless it also provided a way to dynamically change the settings.) What we need is a way to specify either a data view or a string "index pattern", "on the fly", so that the linked UI knows where to find the requested data.

For example, if a Custom Threshold Rule is running in an Elastic deployment, and the rule is using "My Custom Data View" (ID 123xyz) and grouping by host name, each host-related alert would contain links to view the host at /app/metrics/host/detail/{hostname}. If a user clicks that link, the Infra UI will look for that hostname in its configured indices only, which may lead to a 404. If we could instead link to something like /app/metrics/host/detail/{hostname}?data-source=dataview:123xyz or something along those lines, that would be enough to solve the issue.

@neptunian
Copy link
Contributor

@jasonrhodes Thanks. That makes sense. Should be a more straightforward change addressing only supporting it on the fly.

@neptunian neptunian changed the title [Metrics UI] Support using Data views in source configuration [Metrics UI] Support using Data views in UI Mar 20, 2024
@neptunian neptunian changed the title [Metrics UI] Support using Data views in UI [Infra UI] Support using Data views in UI Mar 20, 2024
@roshan-elastic
Copy link

Thanks for this everyone.

Just summarising this in my head, do we know which of the below use cases need data view support to work?

I think the first 3 are going to be the most helpful for users right now...

@neptunian
Copy link
Contributor

The first three could be solved with the "on the fly" data view. The last one involves the user being able to save it to their settings (source config).

@jasonrhodes
Copy link
Member

I agree with what @neptunian said. And just to be super clear, this kind of "on the fly" data source support would need to handle string index patterns OR data views.

@roshan-elastic
Copy link

FYI I've asked a question around the custom threshold rule workflows @neptunian @jasonrhodes

@smith smith added the enhancement New value added to drive a business result label Apr 1, 2024
@smith
Copy link
Contributor

smith commented Jul 10, 2024

This isn't something we're prioritizing at this time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Metrics UI Metrics UI feature needs-refinement A reason and acceptance criteria need to be defined for this issue Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Development

No branches or pull requests

6 participants