Skip to content

Conversation

@mazyu36
Copy link
Contributor

@mazyu36 mazyu36 commented Oct 29, 2025

Newly supported protocol.

https://docs.aws.amazon.com/AWSCloudFormation/latest/TemplateReference/aws-resource-bedrockagentcore-runtime.html#cfn-bedrockagentcore-runtime-protocolconfiguration

Checklist


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

@github-actions github-actions bot added the p2 label Oct 29, 2025
@aws-cdk-automation aws-cdk-automation requested a review from a team October 29, 2025 13:46
@github-actions github-actions bot added the distinguished-contributor [Pilot] contributed 50+ PRs to the CDK label Oct 29, 2025
@krokoko
Copy link
Contributor

krokoko commented Oct 29, 2025

Deployment worked:

with something like

const runtime1 = new Runtime(this, 'MinimalRuntime', {
      runtimeName: 'minimal_runtime',
      agentRuntimeArtifact: agentArtifact,
      protocolConfiguration: ProtocolType.A2A,
    });

One question: should we add a section to the readme to explain the differences and provide links to the documentation ? Depending on the protocol used, there are differences in how the containers should run (see https://docs.aws.amazon.com/bedrock-agentcore/latest/devguide/runtime-a2a.html)

@mazyu36 mazyu36 changed the title chore(bedrock-agentcore): support A2A protocol for runtime chore(bedrock-agentcore): support A2A protocol for runtime Oct 29, 2025
@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 Oct 29, 2025
@mazyu36
Copy link
Contributor Author

mazyu36 commented Oct 29, 2025

Yes, I understand that for a2a, we need to change the port number and path from http on the application side.
However, this is an application-level implementation and isn’t related to the CDK implementation, so I was thinking it’s not something we need to write about here.

What do you think?

@Abogical Abogical self-assigned this Oct 29, 2025
@mergify
Copy link
Contributor

mergify bot commented Oct 29, 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).

@Abogical
Copy link
Member

Abogical commented Oct 29, 2025

If you wish you can add a section in the alpha package readme to explain the difference between those protocols in a separate PR. For now I'll approve this PR.

@mergify
Copy link
Contributor

mergify bot commented Oct 29, 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 01d122c into aws:main Oct 29, 2025
18 of 19 checks passed
@github-actions
Copy link
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 Oct 29, 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 pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants