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

[Inventory] Entity names redirect on click to respective pages #193602

Merged
merged 8 commits into from
Sep 27, 2024

Conversation

Bluefinger
Copy link
Contributor

@Bluefinger Bluefinger commented Sep 20, 2024

Summary

Adds the ability to click through to the overview pages for entities on the Entity Name cell for the Entity Grid on the new Inventory page.

Screen.Recording.2024-09-25.165458.mp4

Closes #192676

How to test

  • Go to Inventory Page.
  • Click on an Entity Name.

Expected: Should redirect to the overview page of that Entity, regardless it is a host, container, or service.

@Bluefinger Bluefinger added release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team v8.16.0 labels Sep 20, 2024
@obltmachine
Copy link

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

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

@Bluefinger
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

merge conflict between base and head

@Bluefinger Bluefinger force-pushed the clickable-entity-names branch from 542525a to 8cd608f Compare September 25, 2024 12:15
@Bluefinger Bluefinger marked this pull request as ready for review September 25, 2024 14:56
@Bluefinger Bluefinger requested a review from a team as a code owner September 25, 2024 14:56
@Bluefinger
Copy link
Contributor Author

@elasticmachine merge upstream

@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Sep 25, 2024
@Bluefinger Bluefinger requested a review from a team as a code owner September 27, 2024 10:34
Copy link
Contributor

@cauemarcondes cauemarcondes left a comment

Choose a reason for hiding this comment

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

LGTM

@Bluefinger
Copy link
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

kibana-ci commented Sep 27, 2024

💛 Build succeeded, but was flaky

  • Buildkite Build
  • Commit: c3606de
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-193602-c3606dec21fa

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #4 / Timeline rendering should trim kqlQueryExpression

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
inventory 200 201 +1

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
observabilityShared 472 473 +1

Async chunks

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

id before after diff
inventory 151.5KB 152.0KB +569.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
inventory 9.8KB 9.9KB +149.0B
observabilityShared 71.1KB 71.1KB +28.0B
total +177.0B
Unknown metric groups

API count

id before after diff
observabilityShared 477 478 +1

History

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

@Bluefinger Bluefinger merged commit 9561698 into elastic:main Sep 27, 2024
25 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11073336108

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 27, 2024
…ic#193602)

## Summary

Adds the ability to click through to the overview pages for entities on
the Entity Name cell for the Entity Grid on the new Inventory page.

https://github.com/user-attachments/assets/e712d3ef-370f-4353-a398-2365176eb582

Closes elastic#192676

### How to test

- Go to Inventory Page.
- Click on an Entity Name.

**Expected**: Should redirect to the overview page of that Entity,
regardless it is a `host`, `container`, or `service`.

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 9561698)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Sep 27, 2024
…193602) (#194315)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Inventory] Entity names redirect on click to respective pages
(#193602)](#193602)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Gonçalo Rica Pais da
Silva","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-27T15:23:12Z","message":"[Inventory]
Entity names redirect on click to respective pages (#193602)\n\n##
Summary\r\n\r\nAdds the ability to click through to the overview pages
for entities on\r\nthe Entity Name cell for the Entity Grid on the new
Inventory
page.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/e712d3ef-370f-4353-a398-2365176eb582\r\n\r\nCloses
#192676 \r\n\r\n### How to test\r\n\r\n- Go to Inventory Page.\r\n-
Click on an Entity Name.\r\n\r\n**Expected**: Should redirect to the
overview page of that Entity,\r\nregardless it is a `host`, `container`,
or `service`.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>","sha":"9561698109fb8382625a58f43ab9e66fd2a1642a","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","ci:project-deploy-observability","Team:obs-ux-infra_services","v8.16.0"],"title":"[Inventory]
Entity names redirect on click to respective
pages","number":193602,"url":"https://github.com/elastic/kibana/pull/193602","mergeCommit":{"message":"[Inventory]
Entity names redirect on click to respective pages (#193602)\n\n##
Summary\r\n\r\nAdds the ability to click through to the overview pages
for entities on\r\nthe Entity Name cell for the Entity Grid on the new
Inventory
page.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/e712d3ef-370f-4353-a398-2365176eb582\r\n\r\nCloses
#192676 \r\n\r\n### How to test\r\n\r\n- Go to Inventory Page.\r\n-
Click on an Entity Name.\r\n\r\n**Expected**: Should redirect to the
overview page of that Entity,\r\nregardless it is a `host`, `container`,
or `service`.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>","sha":"9561698109fb8382625a58f43ab9e66fd2a1642a"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/193602","number":193602,"mergeCommit":{"message":"[Inventory]
Entity names redirect on click to respective pages (#193602)\n\n##
Summary\r\n\r\nAdds the ability to click through to the overview pages
for entities on\r\nthe Entity Name cell for the Entity Grid on the new
Inventory
page.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/e712d3ef-370f-4353-a398-2365176eb582\r\n\r\nCloses
#192676 \r\n\r\n### How to test\r\n\r\n- Go to Inventory Page.\r\n-
Click on an Entity Name.\r\n\r\n**Expected**: Should redirect to the
overview page of that Entity,\r\nregardless it is a `host`, `container`,
or `service`.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic Machine
<[email protected]>","sha":"9561698109fb8382625a58f43ab9e66fd2a1642a"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Gonçalo Rica Pais da Silva <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Inventory] Entity name should be clickable
7 participants