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

[Metrics UI] Enhanced host details - Metadata #80306

Closed
sorantis opened this issue Oct 13, 2020 · 2 comments · Fixed by #84454
Closed

[Metrics UI] Enhanced host details - Metadata #80306

sorantis opened this issue Oct 13, 2020 · 2 comments · Fixed by #84454
Assignees
Labels
enhancement New value added to drive a business result 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

Comments

@sorantis
Copy link

sorantis commented Oct 13, 2020

Add Metadata tab to Enhanced Host Details

image

Properties

Host metadata

  • host.architecture
  • host.hostname
  • host.id
  • host.ip
  • host.mac
  • host.name
  • host.os.build
  • host.os.family
  • host.os.kernel
  • host.os.name
  • host.os.platform
  • host.os.version

Cloud metadata

  • cloud.account.id
  • cloud.account.name
  • cloud.availability_zone
  • cloud.instance.id
  • cloud.instance.name
  • cloud.machine.type
  • cloud.provider
  • cloud.region

Elastic Agent metadata

  • agent.id
  • agent.version
  • agent.policy

Don't show empty values.

The filter icon adds a filter to inventory search field.
Mouse-hove on the filter icon shows the tooltip: Apply filter

@sorantis sorantis added enhancement New value added to drive a business result 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 labels Oct 13, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@hbharding
Copy link
Contributor

hbharding commented Oct 28, 2020

From discussions with the observability designers, we'd like to develop this as a shared component so that other Observability apps can import it into their UIs. This is part of an effort to improve consistency between our UIs. The implementation in our other UIs can be handled in a separate ticket(s) after the component is created.

The information shown below this tab is very similar to what we show in the Logs UI flyout when a user chooses to "view details" for a particular log line. Similarly, APM shows a list of key/values when looking at the metadata for a transaction. We should be able to reuse this new component in places like these.

Logs UI (currently)
image

APM Transaction metadata (currently)
image

Metrics UI (proposed)
image

The proposed changes are as follows:

  1. Uses the compressed form of EuiTable to increase information density. We'll need some custom CSS on the table so that the columns are 33% and 66% wide respectively.
  2. The list of values can optionally be grouped by category headings. For example, "Host", "Cloud", etc.
  3. The values can optionally have a "apply filter" icon button. The filter icon uses $euiColorMediumShadeand $euiColorDarkestShade on hover. The icon's tooltip should say "Apply filter".
  4. Fields with multiple values should only show the first value, but can be toggled to "show more" / "show less". Separating values by a new line (rather than a wrapped, comma separated list as the Logs UI does today) will improve legibility. See GIF for details:

Kapture 2020-10-29 at 16 54 47

@hbharding hbharding changed the title [Metrics UI] Enhanced host details - Properties [Metrics UI] Enhanced host details - Metadata Nov 1, 2020
@phillipb phillipb self-assigned this Nov 16, 2020
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 Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants