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.

Sign up for GitHub

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

Jump to bottom

[8.x] [APM] Fix inventory plugin link (#201122) #201875

Merged
merged 1 commit into from
Nov 26, 2024
Merged

[8.x] [APM] Fix inventory plugin link (#201122) #201875

merged 1 commit into from
Nov 26, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

@cauemarcondes
[APM] Fix inventory plugin link (elastic#201122)
e621559 
closes elastic#201116

- Fix Inventory locator removing `observability` from the url.
- ~~Unify service inventory E2E tests to run it faster avoiding Kibana
loading page on every test.~~
- Add test to navigate to the inventory plugin.

(cherry picked from commit 4bc9a59)
@kibanamachine kibanamachine enabled auto-merge (squash) November 26, 2024 21:59
@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Nov 26, 2024
@kibanamachine kibanamachine mentioned this pull request Nov 26, 2024
Merged
@botelastic botelastic bot added the Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team label Nov 26, 2024
@elasticmachine
Copy link
Contributor

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

@github-actions GitHub Actions
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 docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@kibanamachine kibanamachine merged commit 3426a7f into elastic:8.x Nov 26, 2024
32 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #27 / dashboard app - group 3 dashboard panel copy to "after all" hook for "it always appends new panels instead of overwriting"
  • [job] [logs] FTR Configs #27 / dashboard app - group 3 dashboard panel copy to copies a panel to an existing dashboard

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
inventory 236.3KB 236.3KB -14.0B

Page load bundle

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

id before after diff
observabilityShared 93.7KB 93.7KB -13.0B

cc @cauemarcondes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

Assignees

@cauemarcondes cauemarcondes

Labels
backport ci:project-deploy-observability Create an Observability project Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kibanamachine @elasticmachine @cauemarcondes