Skip to content

[8.19] [Obs AI Assistant] Improve flaky recall tests (#220638) | [Obs AI Assistant] Do not attempt to overwrite index mappings if inference id in component template has changed (#220739)#220791

Merged
sorenlouv merged 2 commits intoelastic:8.19from
sorenlouv:backport/8.19/pr-220638_pr-220739
May 14, 2025

Conversation

@sorenlouv
Copy link
Copy Markdown
Member

sorenlouv added 2 commits May 14, 2025 07:05
…rence id in component template has changed (elastic#220739)

Improve flaky API tests and avoid errors like:

```
[ERROR][plugins.observabilityAIAssistant.service] Failed to PUT mapping for .kibana-observability-ai-assistant-kb: illegal_argument_exception
 	Root causes:
 		illegal_argument_exception: Mapper for [semantic_text] conflicts with existing mapper:
 	Cannot update parameter [inference_id] from [pt_tiny_elser_inference_id] to [pt_tiny_text_embedding_inference_id]
```

(cherry picked from commit d2f25df)
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@sorenlouv sorenlouv merged commit 9ae3b9d into elastic:8.19 May 14, 2025
11 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #6 / CustomFields renders correctly

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
observabilityAIAssistant 401 402 +1
Unknown metric groups

API count

id before after diff
observabilityAIAssistant 403 404 +1

@sorenlouv sorenlouv deleted the backport/8.19/pr-220638_pr-220739 branch May 14, 2025 23:25
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 Team:Obs AI Assistant Observability AI Assistant

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants