Skip to content

Fixes KB recall for inference endpoint connectors#271753

Open
yuliia-fryshko wants to merge 2 commits into
elastic:mainfrom
yuliia-fryshko:obs-ai-assistant-fix-recall-for-inference-connector
Open

Fixes KB recall for inference endpoint connectors#271753
yuliia-fryshko wants to merge 2 commits into
elastic:mainfrom
yuliia-fryshko:obs-ai-assistant-fix-recall-for-inference-connector

Conversation

@yuliia-fryshko
Copy link
Copy Markdown
Contributor

@yuliia-fryshko yuliia-fryshko commented May 28, 2026

Closes https://github.com/elastic/obs-ai-team/issues/638

Fix KB recall for inference endpoint connectors.

Resolve connectors via inference.getConnectorById in the context function and chat recall route instead of actionsClient.get, which only loads action saved objects and fails for eis endpoint ids.

Release note

Fixes knowledge base recall in Observability AI Assistant when the selected connector is an inference endpoint.

@yuliia-fryshko yuliia-fryshko self-assigned this May 28, 2026
@yuliia-fryshko yuliia-fryshko requested a review from a team as a code owner May 28, 2026 15:45
@yuliia-fryshko yuliia-fryshko added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels May 28, 2026
@botelastic botelastic Bot added ci:project-deploy-observability Create an Observability project Team:obs-ai Observability AI team labels May 28, 2026
@infra-vault-gh-plugin-prod
Copy link
Copy Markdown

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!)

@yuliia-fryshko
Copy link
Copy Markdown
Contributor Author

/oblt-deploy

@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented May 28, 2026

💛 Build succeeded, but was flaky

  • Buildkite Build
  • Commit: a5b8c19
  • Build duration: 66 mins
  • Kibana Serverless Image: docker.elastic.co/kibana-ci/kibana-serverless:pr-271753-a5b8c19a5785

Failed CI Steps

Metrics [docs]

✅ unchanged

History

cc @yuliia-fryshko

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown
Contributor

Kibana Exploratory Testing — execution report

Run Metadata

  • Triggered by: Issue #451
  • Elasticsearch image tag: 9.5.0-SNAPSHOT
  • Kibana image: docker.elastic.co/kibana-ci/kibana-serverless:pr-271753-a5b8c19a5785
  • Date: 2026-05-28
  • PR: elastic/kibana#271753 — Fix KB recall for inference endpoint connectors
  • Mode: PR-targeted (journeys generated from PR diff)
  • Journeys executed: 8
  • Passed: 8
  • Errored: 0
  • Findings: 0 bugs, 0 warnings, 0 info

Findings

No findings -- all journeys completed without issues.

Additional details

Screenshots

Screenshots are available in the workflow artifacts.

Workflow run: View workflow run

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.

@yuliia-fryshko Can you please backport this to all affected versions?
And don't skip release notes. Add release_note:fix and mention the release note in a ## Release note section in the PR description. See guidelines here.

@viduni94
Copy link
Copy Markdown
Contributor

@yuliia-fryshko Can you also make sure there are no left over actionsClient implementations after this fix.
Thanks

@neptunian
Copy link
Copy Markdown
Contributor

Is this possible to test?

@viduni94
Copy link
Copy Markdown
Contributor

viduni94 commented May 29, 2026

Is this possible to test?

@neptunian If you are referring to manual tests, I reproduced the error on a QA serverless project, then tested the same query on the serverless deployment on this PR.

Our API tests bypass this error because it always uses the actionsClient for the test and creates an SO beforehand. We should look into how we can update these to use the feature registry instead, now that connectors are deprecated.
cc: @yuliia-fryshko

@yuliia-fryshko yuliia-fryshko added backport:version Backport to applied version labels v9.4.3 labels Jun 1, 2026
@yuliia-fryshko yuliia-fryshko changed the title Fix KB recall for inference endpoint connectors Fixes KB recall for inference endpoint connectors Jun 1, 2026
@yuliia-fryshko yuliia-fryshko added release_note:fix and removed release_note:skip Skip the PR/issue when compiling release notes labels Jun 1, 2026
@yuliia-fryshko
Copy link
Copy Markdown
Contributor Author

/oblt-deploy

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 backport:version Backport to applied version labels ci:project-deploy-observability Create an Observability project release_note:fix Team:obs-ai Observability AI team v9.4.3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants