Skip to content

fix(providers/amazon): Add missing template_fields to BedrockCreateEvaluationJobOperator#68310

Open
ferruzzi wants to merge 1 commit into
apache:mainfrom
aws-mwaa:fix-bedrock-eval-template-fields
Open

fix(providers/amazon): Add missing template_fields to BedrockCreateEvaluationJobOperator#68310
ferruzzi wants to merge 1 commit into
apache:mainfrom
aws-mwaa:fix-bedrock-eval-template-fields

Conversation

@ferruzzi

@ferruzzi ferruzzi commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

The evaluation_config, inference_config, and output_data_config parameters were documented as templated but not included in template_fields. This caused Jinja expressions from XCom to be passed as unrendered strings to the Bedrock API instead of being resolved to their actual values.

This broke the bedrock evaluation system test after PR #68256 changed the bucket name from a hardcoded string to one derived from env_id, resulting in the Bedrock API receiving a Jinja template expression instead of a valid S3 URI.

The evaluation_config, inference_config, and output_data_config parameters
were documented as templated but not included in template_fields.

@SameerMesiah97 SameerMesiah97 left a comment

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.

Trivial Fix. But if its not too much of an incovenience, could you share the screenshots of the failing system test?

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

Labels

area:providers provider:amazon AWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants