Skip to content

[Metrics UI] Custom Metrics for Inventory View - #58072

Merged
simianhacker merged 16 commits into
elastic:masterfrom
simianhacker:inventory-view-custom-metrics-np
Feb 26, 2020
Merged

simianhacker merged 16 commits into
elastic:masterfrom
simianhacker:inventory-view-custom-metrics-np

Conversation

@simianhacker

@simianhacker simianhacker commented Feb 19, 2020 •

Copy link
Copy Markdown
Member

Summary

This PR closes #50193 by adding a custom metric option to the metrics pull down in the Inventory view. This features:

  • Users can create a custom metric using average, max, min, rate aggregations
  • Add new custom metric
  • Edit custom metric
  • Delete custom metric
  • Custom metrics are stored with saved views

This PR also fixes #35006 by widening the custom grouping form and field to be the same width as the custom metrics form.

Screencast

The custom metrics behavior

screencast

Screen Shots

The custom group width matches the custom metrics.

image

Checklist

Delete any items that are not applicable to this PR.

@simianhacker simianhacker added release_note:enhancement Feature:Metrics UI Metrics UI feature v8.0.0 Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v7.7.0 labels Feb 19, 2020
@elasticmachine

Copy link
Copy Markdown
Contributor

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

@simianhacker
simianhacker marked this pull request as ready for review February 21, 2020 14:54
@simianhacker
simianhacker requested a review from a team as a code owner February 21, 2020 14:54
@sorantis

Copy link
Copy Markdown

Looks good! Also +1 for implementing it in a way that persists custom metrics in a saved view.

@kibanamachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@phillipb phillipb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@simianhacker
simianhacker merged commit 8021fd8 into elastic:master Feb 26, 2020
simianhacker added a commit to simianhacker/kibana that referenced this pull request Feb 26, 2020
* [Metrics UI] Add custom metrics interface to Inventory View

* WIP

* Adding workflows for editing custom metrics

* Polishing visual design

* Removing extra text

* Fixing types and return values

* fixing i18n

* Adding aria labels for clearity

* Changing custom group by to match same width as custom metric

* updating integration test for custom metrics

* Fixing type def
simianhacker added a commit that referenced this pull request Feb 26, 2020
* [Metrics UI] Add custom metrics interface to Inventory View

* WIP

* Adding workflows for editing custom metrics

* Polishing visual design

* Removing extra text

* Fixing types and return values

* fixing i18n

* Adding aria labels for clearity

* Changing custom group by to match same width as custom metric

* updating integration test for custom metrics

* Fixing type def
@simianhacker
simianhacker deleted the inventory-view-custom-metrics-np branch April 17, 2024 15:15
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
* [Metrics UI] Add custom metrics interface to Inventory View

* WIP

* Adding workflows for editing custom metrics

* Polishing visual design

* Removing extra text

* Fixing types and return values

* fixing i18n

* Adding aria labels for clearity

* Changing custom group by to match same width as custom metric

* updating integration test for custom metrics

* Fixing type def
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Metrics UI Metrics UI feature release_note:enhancement Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services v7.7.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Metrics UI] Custom Metrics for Inventory View (Accessibility) [Infra UI] Group By custom field not wide enough

5 participants