Skip to content

[9.2] [Agent builder] remove dynamic mapping allocation (#239918)#240224

Merged
kibanamachine merged 1 commit into
elastic:9.2from
kibanamachine:backport/9.2/pr-239918
Oct 23, 2025
Merged

[9.2] [Agent builder] remove dynamic mapping allocation (#239918)#240224
kibanamachine merged 1 commit into
elastic:9.2from
kibanamachine:backport/9.2/pr-239918

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 9.2:

Questions ?

Please refer to the Backport tool documentation

## Summary

- we don't need to index those
- it's dangerous with conversation rounds (different tool results with
same props but different types can lead to mapping exceptions) and with
tool config (same, different tool types can have different types for
their config props)

Because we can't just switch from `dynamic: true` to `dynamic: false`,
and because we don't want to rollout, we introduce a new `config` field
and are doing on-read / on-write migrations to access the legacy field
and move it to the new one

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 1326704)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Oct 23, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) October 23, 2025 09:14
@kibanamachine kibanamachine merged commit 0994632 into elastic:9.2 Oct 23, 2025
16 checks passed
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @pgayvallet

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