Skip to content

[ML] AIOps: Registering embeddables and uiActions synchronously#268001

Merged
jgowdyelastic merged 4 commits into
elastic:mainfrom
jgowdyelastic:register-aiops-embeddables-and-uiactions-synchronously
May 8, 2026
Merged

[ML] AIOps: Registering embeddables and uiActions synchronously#268001
jgowdyelastic merged 4 commits into
elastic:mainfrom
jgowdyelastic:register-aiops-embeddables-and-uiactions-synchronously

Conversation

@jgowdyelastic
Copy link
Copy Markdown
Member

@jgowdyelastic jgowdyelastic commented May 6, 2026

Resolves potential the race conditions by registering embeddables and actions synchronously.
The capabilities check is moved from registration time to the action's isCompatible method and embeddable's buildEmbeddable method.

@rbrtj rbrtj self-requested a review May 6, 2026 20:45
@jgowdyelastic jgowdyelastic self-assigned this May 7, 2026
@jgowdyelastic jgowdyelastic added :ml release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Feature:ML/AIOps ML AIOps features: Change Point Detection, Log Pattern Analysis, Log Rate Analysis v9.5.0 labels May 7, 2026
@jgowdyelastic jgowdyelastic marked this pull request as ready for review May 7, 2026 09:04
@jgowdyelastic jgowdyelastic requested a review from a team as a code owner May 7, 2026 09:04
@infra-vault-gh-plugin-prod
Copy link
Copy Markdown

Pinging @elastic/ml-ui (:ml)

Copy link
Copy Markdown
Contributor

@rbrtj rbrtj left a comment

Choose a reason for hiding this comment

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

LGTM

import { registerCases } from './cases/register_cases';
import { canUseAiops } from './capabilities';

export type AiopsCoreSetup = CoreSetup<AiopsPluginStartDeps, AiopsPluginStart>;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

what do you think of moving it to x-pack/platform/plugins/shared/aiops/public/types/index.ts?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Moved in cc80d2b

@jgowdyelastic jgowdyelastic enabled auto-merge (squash) May 8, 2026 07:58
@kibanamachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Scout Lane #17 - serverless-observability_complete / default / local-serverless-observability_complete - Hosts Page - Flyout - Dashboards Tab
  • [job] [logs] Scout Lane #17 - serverless-observability_complete / default / local-serverless-observability_complete - Hosts Page - Flyout - Logs Tab
  • [job] [logs] Scout Lane #17 - serverless-observability_complete / default / local-serverless-observability_complete - Hosts Page - Flyout - Metadata Tab
  • [job] [logs] Scout Lane #17 - serverless-observability_complete / default / local-serverless-observability_complete - Hosts Page - Flyout - Metrics Tab
  • [job] [logs] Scout Lane #17 - serverless-observability_complete / default / local-serverless-observability_complete - Hosts Page - Flyout - Open as page and return
  • [job] [logs] Scout Lane #17 - serverless-observability_complete / default / local-serverless-observability_complete - Hosts Page - Flyout - Overview Tab - KPI charts and collapsible sections
  • [job] [logs] Scout Lane #17 - serverless-observability_complete / default / local-serverless-observability_complete - Hosts Page - Flyout - Overview Tab - Services section
  • [job] [logs] Scout Lane #17 - serverless-observability_complete / default / local-serverless-observability_complete - Hosts Page - Flyout - Processes Tab
  • [job] [logs] Scout Lane #18 - serverless-observability_complete / default / local-serverless-observability_complete - Serverless Observability Navigation - Complete tier body - clicking body nav items sets the active link, updates breadcrumbs, and navigates
  • [job] [logs] Scout Lane #7 - stateful-classic / default / local-stateful-classic - APM integration not installed but setup completed - Admin user
  • [job] [logs] Scout Lane #7 - stateful-classic / default / local-stateful-classic - Collector integration is not installed - collector integration missing
  • [job] [logs] Scout Lane #7 - stateful-classic / default / local-stateful-classic - Collector integration is not installed - Symbolizer integration is not installed
  • [job] [logs] Scout Lane #7 - stateful-classic / default / local-stateful-classic - Observability Landing Page (discover.isEsqlDefault enabled) - redirects to onboarding when no logs data exists
  • [job] [logs] Scout Lane #7 - stateful-classic / default / local-stateful-classic - Profiling is setup and data is loaded - Admin user
  • [job] [logs] Scout Lane #7 - stateful-classic / default / local-stateful-classic - Profiling is setup and data is loaded - Viewer user

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
aiops 618 619 +1

Async chunks

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

id before after diff
aiops 526.1KB 526.5KB +389.0B

Page load bundle

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

id before after diff
aiops 13.7KB 13.8KB +119.0B

History

cc @jgowdyelastic

@jgowdyelastic jgowdyelastic merged commit 6a1d10b into elastic:main May 8, 2026
46 checks passed
mgadewoll pushed a commit to mgadewoll/kibana that referenced this pull request May 8, 2026
…tic#268001)

Resolves potential the race conditions by registering embeddables and
actions synchronously.
The capabilities check is moved from registration time to the action's
`isCompatible` method and embeddable's `buildEmbeddable` method.
js-jankisalvi pushed a commit to js-jankisalvi/kibana that referenced this pull request May 12, 2026
…tic#268001)

Resolves potential the race conditions by registering embeddables and
actions synchronously.
The capabilities check is moved from registration time to the action's
`isCompatible` method and embeddable's `buildEmbeddable` method.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:ML/AIOps ML AIOps features: Change Point Detection, Log Pattern Analysis, Log Rate Analysis :ml release_note:skip Skip the PR/issue when compiling release notes v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants