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] Remove configurable fields from settings. #41786

Closed
simianhacker opened this issue Jul 23, 2019 · 2 comments
Closed

[Infra UI] Remove configurable fields from settings. #41786

simianhacker opened this issue Jul 23, 2019 · 2 comments
Labels
discuss 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 v8.0.0

Comments

@simianhacker
Copy link
Member

simianhacker commented Jul 23, 2019

Currently in the Infra & Logging UI users can configure the fields we use for host, pod, and containers. This seemed like a good idea at the beginning of the project before ECS was fully implemented. As soon as we realized that the we needed to use kubernetes.pod.uid for the identifier and kubernetes.pod.name for the display name, the configurable fields became problematic. Do we make every field we use configurable? Or should the user massage their data to conform to ECS at ingestion?

The purpose of ECS was to create a schema that enables cross linking and making intelligent decisions about how fields relate to each other. Once we start changing fields then we have to come up with a schema of identifiers on top of ECS; this increases the complexity in favor of configurability and erodes the value of ECS.

I purpose we remove the ability to configure the aforementioned fields and solely rely on ECS. This would be a breaking change and the earliest we could do this is for 8.0, the next major release.

@tbragin @elastic/infra-logs-ui

Related Issues

@weltenwort
Copy link
Member

This also applies to the Logs UI, I would say.

@jasonrhodes
Copy link
Member

At the very least I'd prefer to make this kind of thing a deeper setting that only advanced users are told how to do, rather than leading people into doing it more frequently because we made it so easy. I agree that ECS feels like a really good target to build toward/optimize our curated UI experiences for, provided we don't break the experience for other users.

@sgrodzicki sgrodzicki added Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services [zube]: Backlog labels Jul 29, 2019
@zube zube bot removed the [zube]: Done label Sep 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss 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 v8.0.0
Projects
None yet
Development

No branches or pull requests

4 participants