Skip to content

[9.3] [Obs AI] Remove getRelevantAlertFields from get_alerts tool (#248472)#248552

Merged
viduni94 merged 2 commits intoelastic:9.3from
viduni94:backport/9.3/pr-248472
Jan 10, 2026
Merged

[9.3] [Obs AI] Remove getRelevantAlertFields from get_alerts tool (#248472)#248552
viduni94 merged 2 commits intoelastic:9.3from
viduni94:backport/9.3/pr-248472

Conversation

@viduni94
Copy link
Contributor

@viduni94 viduni94 commented Jan 9, 2026

Backport

This will backport the following commits from main to 9.3:

Questions ?

Please refer to the Backport tool documentation

…tic#248472)

Closes elastic/obs-ai-team#460

## Summary

This PR includes the following updates to the `get_alerts` tool

1. Removed fetching selected fields. These fields don't serve any
purpose at the moment.
2. Removed the system prompt for the tool
3. Filtered the alert document field using `DEFAULT_FIELDS` to reduce
noise.
4. Added new tests and updated existing tests to not use the LLM proxy

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

(cherry picked from commit 63354fe)

# Conflicts:
#	x-pack/solutions/observability/test/api_integration_deployment_agnostic/apis/observability_agent_builder/tools/get_alerts.spec.ts
#	x-pack/solutions/observability/test/api_integration_deployment_agnostic/apis/observability_agent_builder/utils/llm_proxy/scenarios.ts
@viduni94 viduni94 added the backport This PR is a backport of another PR label Jan 9, 2026
@viduni94 viduni94 enabled auto-merge (squash) January 9, 2026 22:34
@viduni94 viduni94 merged commit efdc645 into elastic:9.3 Jan 10, 2026
13 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #34 / EPM Endpoints bulk package upgrade using install api bulk package upgrade with a package already installed "before each" hook for "should upgrade multiple packages"

Metrics [docs]

✅ unchanged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments