Skip to content

fix(bedrock-agentcore-alpha): runtime construct incorrectly forces default lifecycleConfiguration values#36379

Merged
mergify[bot] merged 3 commits intoaws:mainfrom
mazyu36:fix/runtime
Dec 16, 2025
Merged

fix(bedrock-agentcore-alpha): runtime construct incorrectly forces default lifecycleConfiguration values#36379
mergify[bot] merged 3 commits intoaws:mainfrom
mazyu36:fix/runtime

Conversation

@mazyu36
Copy link
Copy Markdown
Contributor

@mazyu36 mazyu36 commented Dec 12, 2025

Issue # (if applicable)

Closes #36376

Reason for this change

The Runtime construct was incorrectly forcing default lifecycleConfiguration values even when users didn't specify them, causing unwanted lifecycle management settings to be applied and diverging from AWS Bedrock AgentCore API behavior where this configuration is completely optional.

Description of changes

Modified to only set lifecycleConfiguration when explicitly provided by users, instead of forcing default values (idleRuntimeSessionTimeout: 900, maxLifetime: 28800)

Describe any new or updated permissions being added

N/A

Description of how you validated changes

  • Updated and verified all existing unit tests pass
  • Updated integration test snapshots that were affected by the removal of default lifecycle configuration

Checklist

BREAKING CHANGE: Runtime constructs will no longer automatically include lifecycleConfiguration with default values when not explicitly specified by users.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@aws-cdk-automation aws-cdk-automation requested a review from a team December 12, 2025 17:36
@github-actions github-actions bot added p2 distinguished-contributor [Pilot] contributed 50+ PRs to the CDK labels Dec 12, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 12, 2025

TestsPassed ✅SkippedFailed
Security Guardian Results130 ran130 passed
TestResult
No test annotations available

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 12, 2025

TestsPassed ✅SkippedFailed
Security Guardian Results with resolved templates130 ran130 passed
TestResult
No test annotations available

@aws-cdk-automation aws-cdk-automation added the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Dec 13, 2025
@alvazjor alvazjor self-assigned this Dec 16, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Dec 16, 2025

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@alvazjor alvazjor changed the title fix(bedrock-agentcore): runtime construct incorrectly forces default lifecycleConfiguration values fix(bedrock-agentcore-alpha): runtime construct incorrectly forces default lifecycleConfiguration values Dec 16, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Dec 16, 2025

Merge Queue Status

✅ The pull request has been merged at 43fd688

This pull request spent 31 minutes 37 seconds in the queue, including 31 minutes 25 seconds running CI.
The checks were run in-place.

Required conditions to merge

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Dec 16, 2025

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit 7954354 into aws:main Dec 16, 2025
26 of 27 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 16, 2025
@aws-cdk-automation aws-cdk-automation removed the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Dec 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

distinguished-contributor [Pilot] contributed 50+ PRs to the CDK p2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(aws-bedrock-agentcore-alpha): Runtime defaults idleRuntimeSessionTimeout to 60s instead of 900s

3 participants