chore(bedrock-alpha): fix failing Bedrock Agent integration tests#37058
Open
aemada-aws wants to merge 2 commits intomainfrom
Open
chore(bedrock-alpha): fix failing Bedrock Agent integration tests#37058aemada-aws wants to merge 2 commits intomainfrom
aemada-aws wants to merge 2 commits intomainfrom
Conversation
Add region constraints to 5 Bedrock Agent integ tests and remove unsupported CodeInterpreter configuration. Root causes: - Tests had no region constraints, deploying to regions without AWS::Bedrock::Agent CFN support or model availability - integ.action-group enabled codeInterpreterEnabled which is incompatible with anthropic.claude-3-5-sonnet-20241022-v2:0 Changes: - Created shared integ-tests-regions.ts with verified region list - Added regions prop to all 5 IntegTest constructs - Removed codeInterpreterEnabled from integ.action-group.ts
Contributor
|
|
||||||||||||||
Contributor
|
|
||||||||||||||
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.
Issue # (if applicable)
N/A
Reason for this change
Five Bedrock Agent integration tests in
aws-bedrock-alphaare failing:integ.action-group:CreateAgentActionGroupRequest caught an error. CodeInterpreter action cannot be specified for model anthropic.claude-3-5-sonnet-20241022-v2:0integ.orchestration:ValidationError: Template format error: Unrecognized resource types: [AWS::Bedrock::Agent]integ.memory:ValidationError: Template format error: Unrecognized resource types: [AWS::Bedrock::Agent]integ.api-schema:ValidationError: Template format error: Unrecognized resource types: [AWS::Bedrock::Agent]integ.agent-existing-role:ValidationError: Template format error: Unrecognized resource types: [AWS::Bedrock::Agent]Root causes:
AWS::Bedrock::AgentCloudFormation resource type or theanthropic.claude-3-5-sonnet-20241022-v2:0model.integ.action-group.tsenabledcodeInterpreterEnabled: truewhich is incompatible with theanthropic.claude-3-5-sonnet-20241022-v2:0model.Description of changes
integ-tests-regions.tswith a verified list of regions where bothAWS::Bedrock::AgentCFN resource type andanthropic.claude-3-5-sonnet-20241022-v2:0model are available. Regions verified viaaws cloudformation describe-typeandaws bedrock get-foundation-model.regionsprop to all 5IntegTestconstructs to constrain deployment to supported regions.Describe any new or updated permissions being added
N/A
Description of how you validated changes
All 5 integration tests pass:
cd packages/@aws-cdk/aws-bedrock-alpha yarn integ \ test/bedrock/agents/integ.action-group.js \ test/bedrock/agents/integ.agent-existing-role.js \ test/bedrock/agents/integ.api-schema.js \ test/bedrock/agents/integ.memory.js \ test/bedrock/agents/integ.orchestration.js \ --disable-update-workflow \ --update-on-failed \ --force \ --parallel-regions us-east-1 us-west-2 us-east-2 ap-northeast-1 ap-southeast-1 ap-southeast-2 ap-south-1 ap-northeast-2 ap-northeast-3 \ --verboseResult: Tests: 5 passed, 5 total
No tests skipped. No tests converted to unit tests.
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license