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] Amend inventory title to "Infrastructure Inventory" #193189

Merged
merged 9 commits into from
Sep 20, 2024

Conversation

Bluefinger
Copy link
Contributor

@Bluefinger Bluefinger commented Sep 17, 2024

Summary

Update the Inventory title terminology to be "Infrastructure Inventory", in order to differentiate from the incoming top-level "Inventory" page.

Screenshot 2024-09-17 170745

Closes #192324

How to test

  • Load Elastic cluster, go to Infrastructure -> Infrastructure Inventory

Expected: The side menu link text should match the breadcrumb title and the main title text for the page.

@Bluefinger Bluefinger added release_note:skip Skip the PR/issue when compiling release notes backport:skip This commit does not require backporting Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Sep 17, 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 Bluefinger marked this pull request as ready for review September 18, 2024 09:00
@Bluefinger Bluefinger requested review from a team as code owners September 18, 2024 09:00
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Sep 18, 2024
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

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. Maybe you could reuse the label already defined on translations.ts?

@crespocarlos crespocarlos added backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) and removed backport:skip This commit does not require backporting labels Sep 19, 2024
Copy link
Contributor

@crespocarlos crespocarlos left a comment

Choose a reason for hiding this comment

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

We still need to change it here:

image

@kibana-ci
Copy link
Collaborator

kibana-ci commented Sep 19, 2024

💛 Build succeeded, but was flaky

  • Buildkite Build
  • Commit: 238d317
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-193189-238d317c9c23

Failed CI Steps

Metrics [docs]

Async chunks

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

id before after diff
infra 1.6MB 1.6MB -1.4KB

Page load bundle

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

id before after diff
infra 51.1KB 52.4KB +1.3KB

History

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

Copy link
Contributor

@yngrdyn yngrdyn 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 Bluefinger merged commit 04c3486 into elastic:main Sep 20, 2024
23 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 20, 2024
…193189)

## Summary

Update the Inventory title terminology to be "Infrastructure Inventory",
in order to differentiate from the incoming top-level "Inventory" page.

<img width="481" alt="Screenshot 2024-09-17 170745"
src="https://github.com/user-attachments/assets/d72dfb6e-fe3b-44d8-b6f0-ae333586ceb4">

Closes elastic#192324

# How to test

- Load Elastic cluster, go to Infrastructure -> Infrastructure Inventory

**Expected**: The side menu link text should match the breadcrumb title
and the main title text for the page.

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit 04c3486)
@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 20, 2024
…&quot; (#193189) (#193523)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Infra] Amend inventory title to &quot;Infrastructure Inventory&quot;
(#193189)](#193189)

<!--- 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-20T09:41:47Z","message":"[Infra]
Amend inventory title to \"Infrastructure Inventory\" (#193189)\n\n##
Summary\r\n\r\nUpdate the Inventory title terminology to be
\"Infrastructure Inventory\",\r\nin order to differentiate from the
incoming top-level \"Inventory\" page.\r\n\r\n<img width=\"481\"
alt=\"Screenshot 2024-09-17
170745\"\r\nsrc=\"https://github.com/user-attachments/assets/d72dfb6e-fe3b-44d8-b6f0-ae333586ceb4\">\r\n\r\nCloses
#192324 \r\n\r\n# How to test\r\n\r\n- Load Elastic cluster, go to
Infrastructure -> Infrastructure Inventory\r\n\r\n**Expected**: The side
menu link text should match the breadcrumb title\r\nand the main title
text for the page.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic
Machine
<[email protected]>","sha":"04c3486878efed32e99fd4d17c893f69a10ed9ab","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"],"title":"[Infra]
Amend inventory title to \"Infrastructure
Inventory\"","number":193189,"url":"https://github.com/elastic/kibana/pull/193189","mergeCommit":{"message":"[Infra]
Amend inventory title to \"Infrastructure Inventory\" (#193189)\n\n##
Summary\r\n\r\nUpdate the Inventory title terminology to be
\"Infrastructure Inventory\",\r\nin order to differentiate from the
incoming top-level \"Inventory\" page.\r\n\r\n<img width=\"481\"
alt=\"Screenshot 2024-09-17
170745\"\r\nsrc=\"https://github.com/user-attachments/assets/d72dfb6e-fe3b-44d8-b6f0-ae333586ceb4\">\r\n\r\nCloses
#192324 \r\n\r\n# How to test\r\n\r\n- Load Elastic cluster, go to
Infrastructure -> Infrastructure Inventory\r\n\r\n**Expected**: The side
menu link text should match the breadcrumb title\r\nand the main title
text for the page.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic
Machine
<[email protected]>","sha":"04c3486878efed32e99fd4d17c893f69a10ed9ab"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/193189","number":193189,"mergeCommit":{"message":"[Infra]
Amend inventory title to \"Infrastructure Inventory\" (#193189)\n\n##
Summary\r\n\r\nUpdate the Inventory title terminology to be
\"Infrastructure Inventory\",\r\nin order to differentiate from the
incoming top-level \"Inventory\" page.\r\n\r\n<img width=\"481\"
alt=\"Screenshot 2024-09-17
170745\"\r\nsrc=\"https://github.com/user-attachments/assets/d72dfb6e-fe3b-44d8-b6f0-ae333586ceb4\">\r\n\r\nCloses
#192324 \r\n\r\n# How to test\r\n\r\n- Load Elastic cluster, go to
Infrastructure -> Infrastructure Inventory\r\n\r\n**Expected**: The side
menu link text should match the breadcrumb title\r\nand the main title
text for the page.\r\n\r\n---------\r\n\r\nCo-authored-by: Elastic
Machine
<[email protected]>","sha":"04c3486878efed32e99fd4d17c893f69a10ed9ab"}}]}]
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.

[Infra] Rename infra "Inventory" to "Infrastructure Inventory"
9 participants