[AO] use toMountPoint from react-kibana-mount for header menu portal

[mgiota]

Fixes #164366

Acceptance Criteria

• use toMountPoint from @kbn/react-kibana-mount for kibana/x-pack/plugins/observability/public/pages/overview/components/header_menu/header_menu_portal.tsx

[apmmachine]

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

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

[elasticmachine]

Pinging @elastic/actionable-observability (Team: Actionable Observability)

[mgiota]

@elasticmachine merge upstream

[mgiota]

@elasticmachine merge upstream

[mgiota]

@elasticmachine merge upstream

[mgiota]

@simianhacker This is probably out of scope of this PR, but while working on this one, I noticed there is an observability shared HeaderMenuPortal component, which we could eventually use at some point.

import { HeaderMenuPortal } from '@kbn/observability-shared-plugin/public';

However, the shared plugin still uses toMountPoint from the old path '@kbn/kibana-react-plugin/public'. I see observability-ui team is responsible for making changes under observability-shared-plugin, right? Once they update using the proper toMountPoint, we could reuse the shared HeaderPortalMenu. Shall I create a ticket for our team?

[simianhacker]

I see observability-ui team is responsible for making changes under observability-shared-plugin, right? Once they update using the proper toMountPoint, we could reuse the shared HeaderPortalMenu. Shall I create a ticket for our team?

I'm not familiar with the HeaderPortalMenu... maybe we should also update the path in that component. Then expore using it, if that makes sense.

[mgiota]

@simianhacker Yes I could do this. Here are the places that make use of the shared HeaderMenuPortal.

[kibana-ci]

💚 Build Succeeded

• Buildkite Build
• Commit: 233dd72

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id	before	after	diff
observability	457	463	+6

Async chunks

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

id	before	after	diff
observability	1.0MB	1.0MB	+717.0B

Unknown metric groups

References to deprecated APIs

id	before	after	diff
observability	40	38	-2

History

• 💚 Build #163122 succeeded 95ddfde
• 💔 Build #162290 failed 82caa86
• 💔 Build #162249 failed 92b885e
• 💔 Build #162221 failed c64f1ad
• 💔 Build #162067 failed 1d695a1
• 💔 Build #162063 failed 9b793a7

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

cc @mgiota

[simianhacker]

LGTM