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

[Infrastructure UI] Hosts Flyout: Add pins to the metadata table #155904

Conversation

jennypavlova
Copy link
Member

Closes #155190

Summary

@jennypavlova jennypavlova added Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting labels Apr 26, 2023
@jennypavlova jennypavlova self-assigned this Apr 26, 2023
@apmmachine
Copy link
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@jennypavlova jennypavlova changed the title Add pins [Infrastructure UI] Hosts Flyout: Add pins to the metadata table Apr 26, 2023
Test add/persist/remove pin
@jennypavlova jennypavlova force-pushed the 155190-infrastructure-ui-hosts-flyout-add-pins-to-the-metadata-table branch from 35aaadb to ce499af Compare April 26, 2023 18:04
@kibana-ci
Copy link
Collaborator

kibana-ci commented Apr 26, 2023

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #34 / InfraOps App Metrics UI Hosts View #Page Content Metrics Tab should have an option to open the chart in lens
  • [job] [logs] FTR Configs #48 / InfraOps App Metrics UI Hosts View #Page Content Metrics Tab should have an option to open the chart in lens
  • [job] [logs] FTR Configs #34 / InfraOps App Metrics UI Hosts View #Page Content Metrics Tab should have an option to open the chart in lens
  • [job] [logs] FTR Configs #48 / InfraOps App Metrics UI Hosts View #Page Content Metrics Tab should have an option to open the chart in lens
  • [job] [logs] FTR Configs #34 / InfraOps App Metrics UI Hosts View #Page Content Metrics Tab should load 8 lens metric charts
  • [job] [logs] FTR Configs #48 / InfraOps App Metrics UI Hosts View #Page Content Metrics Tab should load 8 lens metric charts
  • [job] [logs] FTR Configs #34 / InfraOps App Metrics UI Hosts View #Page Content Metrics Tab should load 8 lens metric charts
  • [job] [logs] FTR Configs #48 / InfraOps App Metrics UI Hosts View #Page Content Metrics Tab should load 8 lens metric charts
  • [job] [logs] FTR Configs #34 / InfraOps App Metrics UI Hosts View #Page Content Search Query should filter the table content on a search submit
  • [job] [logs] FTR Configs #48 / InfraOps App Metrics UI Hosts View #Single host Flyout should render metadata tab, pin and unpin table row

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
infra 1357 1358 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
infra 2.0MB 2.0MB +1.6KB
Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 17 19 +2
securitySolution 396 399 +3
total +5

Total ESLint disabled count

id before after diff
enterpriseSearch 18 20 +2
securitySolution 476 479 +3
total +5

History

  • 💔 Build #123585 failed 35aaadb279371067478c594f12272bb22fd88738

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

cc @jennypavlova

@jennypavlova
Copy link
Member Author

#155190 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes 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 this pull request may close these issues.

[Infrastructure UI] Asset Details: Add pins to the metadata table
3 participants