Skip to content

[Obs AI Assistant] Disable in Logs Essentials mode#223236

Merged
sorenlouv merged 9 commits intoelastic:mainfrom
sorenlouv:disable-obs-ai-assistant-in-logs-essential
Jun 20, 2025
Merged

[Obs AI Assistant] Disable in Logs Essentials mode#223236
sorenlouv merged 9 commits intoelastic:mainfrom
sorenlouv:disable-obs-ai-assistant-in-logs-essential

Conversation

@sorenlouv
Copy link
Copy Markdown
Contributor

@sorenlouv sorenlouv commented Jun 10, 2025 •
edited by kibanamachine
Loading

Closes https://github.com/elastic/observability-dev/issues/4498
Closes https://github.com/elastic/obs-ai-assistant-team/issues/257

This PR disables the Obs AI Assistant when running in logs essential mode:

Testing steps

  1. Add the following to kibana.dev.yml:
pricing.tiers.enabled: true
pricing.tiers.products:
  - name: observability
    tier: logs_essentials
  1. Start Kibana and ES in Serverless mode:
yarn start --serverless=oblt
yarn es serverless --projectType=oblt
  1. Verify that AI Assistant is not accessible.
  2. ?
  3. Profit

Blocked by: #223875

@sorenlouv sorenlouv requested review from a team as code owners June 10, 2025 13:17
@sorenlouv sorenlouv added release_note:skip Skip the PR/issue when compiling release notes v9.1.0 v8.19.0 labels Jun 10, 2025
@sorenlouv sorenlouv enabled auto-merge (squash) June 10, 2025 17:16
@sorenlouv sorenlouv requested review from a team as code owners June 11, 2025 07:55
tonyghiani
tonyghiani reviewed Jun 11, 2025
View reviewed changes
Comment thread src/platform/packages/shared/serverless/settings/observability_project/index.ts Outdated
@bryce-b bryce-b mentioned this pull request Jun 11, 2025
Merged
1 task
dominiqueclarke
dominiqueclarke approved these changes Jun 12, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@dominiqueclarke dominiqueclarke left a comment

Choose a reason for hiding this comment

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

obs-ux-management changes LGTM. If this is merged before custom roles, we may need to push another commit cc: @jasonrhodes

@sorenlouv sorenlouv requested a review from a team as a code owner June 16, 2025 09:31
@botelastic botelastic Bot added ci:project-deploy-observability Create an Observability project Team:Obs AI Assistant Observability AI Assistant labels Jun 16, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jun 16, 2025

Pinging @elastic/obs-ai-assistant (Team:Obs AI Assistant)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 16, 2025

🤖 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
[CI] Auto-commit changed files from 'ts-node .buildkite/pipeline-reso…
091e076 
…urce-definitions/scripts/fix-location-collection.ts'
@sorenlouv sorenlouv mentioned this pull request Jun 16, 2025
Open
@sorenlouv sorenlouv added the backport:version Backport to applied version labels label Jun 17, 2025
@sorenlouv
Copy link
Copy Markdown
Contributor Author

sorenlouv commented Jun 18, 2025

@rudolf I applied your suggestion. Mind taking a look, and approve if it looks good to you?

@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jun 20, 2025 •
edited
Loading

💚 Build Succeeded

  • Buildkite Build
  • Commit: 30cba0f
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-223236-30cba0fcbdbb

Metrics [docs]

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
@kbn/serverless-observability-settings 1 2 +1
Unknown metric groups

API count

id before after diff
@kbn/serverless-observability-settings 1 2 +1

History

@sorenlouv sorenlouv merged commit 49d366c into elastic:main Jun 20, 2025
11 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Jun 20, 2025

Starting backport for target branches: 8.19

https://github.com/elastic/kibana/actions/runs/15777235246

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 20, 2025
@sorenlouv
[Obs AI Assistant] Disable in Logs Essentials mode (elastic#223236)
0ac9d9c 
Closes https://github.com/elastic/observability-dev/issues/4498
Closes https://github.com/elastic/obs-ai-assistant-team/issues/257

(cherry picked from commit 49d366c)
@kibanamachine kibanamachine mentioned this pull request Jun 20, 2025
Merged
@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Jun 20, 2025

💚 All backports created successfully

Status Branch Result
✅ 8.19

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 Jun 20, 2025
@kibanamachine @sorenlouv
[8.19] [Obs AI Assistant] Disable in Logs Essentials mode (#223236) (#…
988e5c8 
…224662)

# Backport

This will backport the following commits from `main` to `8.19`:
- [[Obs AI Assistant] Disable in Logs Essentials mode
(#223236)](#223236)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Søren
Louv-Jansen","email":"soren.louv@elastic.co"},"sourceCommit":{"committedDate":"2025-06-20T10:47:25Z","message":"[Obs
AI Assistant] Disable in Logs Essentials mode (#223236)\n\nCloses
https://github.com/elastic/observability-dev/issues/4498\nCloses
https://github.com/elastic/obs-ai-assistant-team/issues/257","sha":"49d366c70ac02c79d0f547d8683983413575731f","branchLabelMapping":{"^v9.1.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Obs
AI
Assistant","ci:project-deploy-observability","backport:version","v9.1.0","v8.19.0"],"title":"[Obs
AI Assistant] Disable in Logs Essentials
mode","number":223236,"url":"https://github.com/elastic/kibana/pull/223236","mergeCommit":{"message":"[Obs
AI Assistant] Disable in Logs Essentials mode (#223236)\n\nCloses
https://github.com/elastic/observability-dev/issues/4498\nCloses
https://github.com/elastic/obs-ai-assistant-team/issues/257","sha":"49d366c70ac02c79d0f547d8683983413575731f"}},"sourceBranch":"main","suggestedTargetBranches":["8.19"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/223236","number":223236,"mergeCommit":{"message":"[Obs
AI Assistant] Disable in Logs Essentials mode (#223236)\n\nCloses
https://github.com/elastic/observability-dev/issues/4498\nCloses
https://github.com/elastic/obs-ai-assistant-team/issues/257","sha":"49d366c70ac02c79d0f547d8683983413575731f"}},{"branch":"8.19","label":"v8.19.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Søren Louv-Jansen <soren.louv@elastic.co>
akowalska622 pushed a commit to akowalska622/kibana that referenced this pull request Jun 25, 2025
@sorenlouv @akowalska622
[Obs AI Assistant] Disable in Logs Essentials mode (elastic#223236)
304b7fb 
Closes elastic/observability-dev#4498
Closes elastic/obs-ai-team#257
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sabarasaba sabarasaba sabarasaba approved these changes

@rudolf rudolf rudolf approved these changes

@Dosant Dosant Dosant approved these changes

@dominiqueclarke dominiqueclarke dominiqueclarke approved these changes

@arturoliduena arturoliduena arturoliduena approved these changes

@tonyghiani tonyghiani tonyghiani approved these changes

Assignees

No one assigned

Labels

backport:version Backport to applied version labels ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:Obs AI Assistant Observability AI Assistant v8.19.0 v9.1.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

@sorenlouv @elasticmachine @kibanamachine @sabarasaba @rudolf @Dosant @dominiqueclarke @arturoliduena @tonyghiani