Skip to content

Comments

chore(bedrock-alpha): fix failing Bedrock Agent integration tests#37058

Open
aemada-aws wants to merge 2 commits intomainfrom
fix/bedrock-alpha-integ-tests-remediation
Open

chore(bedrock-alpha): fix failing Bedrock Agent integration tests#37058
aemada-aws wants to merge 2 commits intomainfrom
fix/bedrock-alpha-integ-tests-remediation

Conversation

@aemada-aws
Copy link
Contributor

@aemada-aws aemada-aws commented Feb 23, 2026

Issue # (if applicable)

N/A

Reason for this change

Five Bedrock Agent integration tests in aws-bedrock-alpha are failing:

  1. integ.action-group: CreateAgentActionGroupRequest caught an error. CodeInterpreter action cannot be specified for model anthropic.claude-3-5-sonnet-20241022-v2:0
  2. integ.orchestration: ValidationError: Template format error: Unrecognized resource types: [AWS::Bedrock::Agent]
  3. integ.memory: ValidationError: Template format error: Unrecognized resource types: [AWS::Bedrock::Agent]
  4. integ.api-schema: ValidationError: Template format error: Unrecognized resource types: [AWS::Bedrock::Agent]
  5. integ.agent-existing-role: ValidationError: Template format error: Unrecognized resource types: [AWS::Bedrock::Agent]

Root causes:

  • Tests had no region constraints, causing deployment to regions that don't support the AWS::Bedrock::Agent CloudFormation resource type or the anthropic.claude-3-5-sonnet-20241022-v2:0 model.
  • integ.action-group.ts enabled codeInterpreterEnabled: true which is incompatible with the anthropic.claude-3-5-sonnet-20241022-v2:0 model.

Description of changes

  • Created shared integ-tests-regions.ts with a verified list of regions where both AWS::Bedrock::Agent CFN resource type and anthropic.claude-3-5-sonnet-20241022-v2:0 model are available. Regions verified via aws cloudformation describe-type and aws bedrock get-foundation-model.
  • Added regions prop to all 5 IntegTest constructs 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 \
  --verbose

Result: Tests: 5 passed, 5 total

  • integ.orchestration → us-east-1: SUCCESS
  • integ.agent-existing-role → ap-northeast-1: SUCCESS
  • integ.memory → us-west-2: SUCCESS
  • integ.action-group → ap-southeast-1: SUCCESS
  • integ.api-schema → us-east-2: SUCCESS

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

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
@github-actions github-actions bot added the p2 label Feb 23, 2026
@aemada-aws aemada-aws changed the title fix(aws-bedrock-alpha): fix failing Bedrock Agent integration tests chore(bedrock-alpha): fix failing Bedrock Agent integration tests Feb 23, 2026
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Feb 23, 2026
@aemada-aws aemada-aws added the pr/needs-integration-tests-deployment Requires the PR to deploy the integration test snapshots. label Feb 23, 2026
@aemada-aws aemada-aws had a problem deploying to deployment-integ-test February 23, 2026 13:25 — with GitHub Actions Failure
@github-actions
Copy link
Contributor

github-actions bot commented Feb 23, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results48 ran48 passed
TestResult
No test annotations available

@github-actions
Copy link
Contributor

github-actions bot commented Feb 23, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results with resolved templates48 ran48 passed
TestResult
No test annotations available

@aws-cdk-automation aws-cdk-automation added the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS. p2 pr/needs-integration-tests-deployment Requires the PR to deploy the integration test snapshots. pr/needs-maintainer-review This PR needs a review from a Core Team Member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants