Skip to content

Conversation

@alvazjor
Copy link
Contributor

Issue # (if applicable)

Closes #35968

Reason for this change

Using AgentRuntimeArtifact.fromAsset() adds an MD5 hash of the directory path to the construct ID, causing CDK pipelines to self-mutate whenever the Docker image directory contents change. This is inconsistent with how other Docker images handle assets.

Description of changes

  • Removed the md5hash import and usage from AssetImage.bind() and changed the construct ID from AgentRuntimeArtifact${hash} to a static AgentRuntimeArtifact
  • Added unit tests to verify the static ID behavior
  • Fixed existing test paths to use path.join(__dirname, ...) for portability

Describe any new or updated permissions being added

NA

Description of how you validated changes

Added unit tests

Checklist


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 November 28, 2025 18:12
@github-actions github-actions bot added bug This issue is a bug. effort/medium Medium work item – several days of effort p2 labels Nov 28, 2025
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Nov 28, 2025
Copy link
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(This review is outdated)

@alvazjor alvazjor added the pr-linter/exempt-integ-test The PR linter will not require integ test changes label Nov 28, 2025
@aws-cdk-automation aws-cdk-automation dismissed their stale review November 28, 2025 18:19

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

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

Labels

bug This issue is a bug. contribution/core This is a PR that came from AWS. effort/medium Medium work item – several days of effort p2 pr/needs-maintainer-review This PR needs a review from a Core Team Member pr-linter/exempt-integ-test The PR linter will not require integ test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(bedrock-agentcore-alpha): image asset based runtime artifacts cause pipeline self-mutation

2 participants