Skip to content

[aws_bedrock] Map Converse API nested inference parameters and input/output messages to gen_ai fields. - #20868

Open
muskan-agarwal26 wants to merge 2 commits into
elastic:mainfrom
muskan-agarwal26:enhancement/aws_bedrock-1.8.0
Open

[aws_bedrock] Map Converse API nested inference parameters and input/output messages to gen_ai fields.#20868
muskan-agarwal26 wants to merge 2 commits into
elastic:mainfrom
muskan-agarwal26:enhancement/aws_bedrock-1.8.0

Conversation

@muskan-agarwal26

Copy link
Copy Markdown
Contributor

Type of change

  • Enhancement

Proposed commit message

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

How to test this PR locally

elastic-package test

Related issues

@muskan-agarwal26
muskan-agarwal26 requested review from a team as code owners August 24, 2026 09:00
@muskan-agarwal26 muskan-agarwal26 self-assigned this Aug 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ Elastic Docs Style Checker (Vale)

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@efd6

efd6 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

/test

Comment on lines +140 to +145
- name: gen_ai.input.messages
type: flattened
description: Chat history provided to the model as input.
- name: gen_ai.output.messages
type: flattened
description: Messages returned by the model for this invocation.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
- name: gen_ai.input.messages
type: flattened
description: Chat history provided to the model as input.
- name: gen_ai.output.messages
type: flattened
description: Messages returned by the model for this invocation.
- name: gen_ai.input.messages
external: ecs
- name: gen_ai.output.messages
external: ecs

Comment on lines +526 to +531
if (ctx.gen_ai == null) {
ctx.gen_ai = new HashMap();
}
if (ctx.gen_ai.output == null) {
ctx.gen_ai.output = new HashMap();
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
if (ctx.gen_ai == null) {
ctx.gen_ai = new HashMap();
}
if (ctx.gen_ai.output == null) {
ctx.gen_ai.output = new HashMap();
}
ctx.gen_ai = ctx.gen_ai ?: [:];
ctx.gen_ai.output = ctx.gen_ai.output ?: [:];

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

✅ All changelog entries have the correct PR link.

@infra-vault-gh-plugin-prod

infra-vault-gh-plugin-prod Bot commented Aug 25, 2026

Copy link
Copy Markdown

💔 Build Failed

Failed CI Steps

History

cc @muskan-agarwal26

@efd6

efd6 commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

This is failing because 8.x flattened fields do not allow indexing of arrays of objects. The behaviour for 9.x appears to be that this is allowed. I could not find a way to reshape the fields so that 8.16.5 (or 8.19.11) would accept the documents. If you can find something that works, please do, otherwise we may need to drop support for 8.x

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

aws_bedrock: the invocation data stream fails to map some informative fields

2 participants