fix(providers/amazon): Add missing template_fields to BedrockCreateEvaluationJobOperator#68310
Open
ferruzzi wants to merge 1 commit into
Open
fix(providers/amazon): Add missing template_fields to BedrockCreateEvaluationJobOperator#68310ferruzzi wants to merge 1 commit into
ferruzzi wants to merge 1 commit into
Conversation
The evaluation_config, inference_config, and output_data_config parameters were documented as templated but not included in template_fields.
1 task
SameerMesiah97
approved these changes
Jun 10, 2026
SameerMesiah97
left a comment
Contributor
There was a problem hiding this comment.
Trivial Fix. But if its not too much of an incovenience, could you share the screenshots of the failing system test?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
evaluation_config,inference_config, andoutput_data_configparameters were documented as templated but not included intemplate_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.