Skip to content

[AI Infra] Chat Experience: hide Observability and Elasticsearch AI Assistant side bar setting on serverless#246534

Merged
KodeRad merged 6 commits intoelastic:mainfrom
KodeRad:ml-chat-experience-hide-oblt-es-ai-assistant-side-bar
Dec 16, 2025
Merged

[AI Infra] Chat Experience: hide Observability and Elasticsearch AI Assistant side bar setting on serverless#246534
KodeRad merged 6 commits intoelastic:mainfrom
KodeRad:ml-chat-experience-hide-oblt-es-ai-assistant-side-bar

Conversation

@KodeRad
Copy link
Copy Markdown
Contributor

@KodeRad KodeRad commented Dec 16, 2025

Summary

Follow up to: #244532

This PR:

  • Hides AI Assistant side bar setting when Chat Experience is set to Agent
    • for Elasticsearch Serverless
    • for Observability Serverless
  • fixes the issue with AI Assistant button in nav bar on Elasticsearch serverless
    • it threw use_agent_builder_opt_in.ts:24 Uncaught TypeError: Cannot read properties of undefined (reading 'onechat') becasue onechat plugin was not available at the time

Summarize your PR. If it involves visual changes include a screenshot or gif.

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

  • Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
  • Documentation was added for features that require explanation or tutorials
  • Unit or functional tests were updated or added to match the most common scenarios
  • If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
  • This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The release_note:breaking label should be applied in these situations.
  • Flaky Test Runner was used on any tests changed
  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.

Identify risks

Does this PR introduce any risks? For example, consider risks like hard to test bugs, performance regression, potential of data loss.

Describe the risk, its severity, and mitigation for each identified risk. Invite stakeholders and evaluate how to proceed before merging.

@KodeRad KodeRad self-assigned this Dec 16, 2025
@KodeRad KodeRad added v9.3.0 :ml Team:ML Team label for ML (also use :ml) t// Team:AI Infra Platform AppEx AI Infrastructure Team t// release_note:fix backport:skip This PR does not require backporting labels Dec 16, 2025
@KodeRad KodeRad marked this pull request as ready for review December 16, 2025 12:15
@KodeRad KodeRad requested review from a team as code owners December 16, 2025 12:15
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/appex-ai-infra (Team:AI Infra)

@botelastic botelastic Bot added ci:project-deploy-observability Create an Observability project Team:obs-ai Observability AI team labels Dec 16, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ai-team (Team:obs-ai)

@github-actions
Copy link
Copy Markdown
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!)

@KodeRad KodeRad requested a review from viduni94 December 16, 2025 12:36
Copy link
Copy Markdown
Member

@jennypavlova jennypavlova left a comment

Choose a reason for hiding this comment

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

Obs presentation changes LGTM

Copy link
Copy Markdown
Contributor

@viduni94 viduni94 left a comment

Choose a reason for hiding this comment

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

Thank you @KodeRad
LGTM

@KodeRad KodeRad enabled auto-merge (squash) December 16, 2025 14:00
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Dec 16, 2025

💛 Build succeeded, but was flaky

  • Buildkite Build
  • Commit: f6acb2d
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-246534-f6acb2d649fc

Failed CI Steps

Test Failures

  • [job] [logs] Scout: [ platform / discover_enhanced ] plugin / dont show up if outside of range
  • [job] [logs] Scout: [ platform / discover_enhanced ] plugin / serverless-es - Discover app - value suggestions: useTimeRange enabled - dont show up if outside of range
  • [job] [logs] Jest Integration Tests #3 / Uprade package install version with package installed with a previous format install version should upgrade package install version for outdated packages

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
serverlessSearch 176 184 +8

Page load bundle

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

id before after diff
observabilityAIAssistant 52.9KB 52.9KB +43.0B
serverlessObservability 18.1KB 18.1KB +2.0B
serverlessSearch 23.4KB 23.6KB +261.0B
total +306.0B

History

cc @KodeRad

@KodeRad KodeRad added the release_note:skip Skip the PR/issue when compiling release notes label Dec 16, 2025
@KodeRad KodeRad merged commit b081ba0 into elastic:main Dec 16, 2025
16 checks passed
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 ci:project-deploy-observability Create an Observability project :ml release_note:skip Skip the PR/issue when compiling release notes Team:AI Infra Platform AppEx AI Infrastructure Team t// Team:ML Team label for ML (also use :ml) t// Team:obs-ai Observability AI team v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants