Skip to content

Conversation

neylsoncrepalde
Copy link

@neylsoncrepalde neylsoncrepalde commented Aug 18, 2025

Amazon Bedrock AgentCore Samples Pull Request

Instructions

  • Do not erase any parts of this template that are not applicable to your pull request.
  • If a section is not applicable, explicitly state the reason.
    • Tick the checkboxes for the items you have completed.
  • These are mandatory requirements, not mere suggestions.

Describe your changes

  • Implementation of a new example on the 01-tutorials/01-AgentCorte-runtime/03-advanced-concepts. The example shows how to implement a custom container an AgentCore runtime agent with streaming response.
No changes. New tutorial.

Issue ticket number and link

  • Issue # (if applicable)

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Are you uploading a dataset?
  • Have you added contributions to RELEASE NOTES?

New Example Submissions:

  • Have you tested your code, and made sure the functionality runs successfully? Provide screenshots.
  • Have you linted your Python code with black?
  • Have you documented Introduction, Architecture Diagram, Prerequisites, Usage, Sample Prompts, and Clean Up steps in your example README?
  • I agree to resolve any issues created for this example in the future.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@@ -0,0 +1,644 @@
{
Copy link
Contributor

@w601sxs w601sxs Aug 18, 2025

Choose a reason for hiding this comment

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

do we need this, or can we build using the CLI? looks like the script is pretty std?


Reply via ReviewNB

Copy link
Author

Choose a reason for hiding this comment

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

Actually, I looked for a way to build a custom container with the agentcore CLI and could not do it. In this case, we have to do it manually because we are dealing with multiple agent files.

@@ -0,0 +1,644 @@
{
Copy link
Contributor

@w601sxs w601sxs Aug 18, 2025

Choose a reason for hiding this comment

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

Line #3.    client = boto3.client('bedrock-agentcore-control')

can you see if python SDK works? Instead of bot3


Reply via ReviewNB

Copy link
Author

Choose a reason for hiding this comment

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

I checked and didn't find a way to deploy a custom container agent with the SDK, only an agent built with a single .py file and using the SDK's decorators. This is why we are using boto3.

@@ -0,0 +1,644 @@
{
Copy link
Contributor

@w601sxs w601sxs Aug 18, 2025

Choose a reason for hiding this comment

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

Line #5.    agent_core_client = boto3.client('bedrock-agentcore', region_name='us-east-1')

this can still be boto3, unlss you think there is value in doing everything from scratch, pelase do comment


Reply via ReviewNB

Copy link
Author

Choose a reason for hiding this comment

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

This is the same case as the last comment. I did not find a way to make a call to a custom container agent through agentcore SDK. Only for agents with single .py files built with the SDK decorators.

@EashanKaushik EashanKaushik added 01-tutorials 01-tutorials 01-AgentCore-runtime 01-tutorials/01-AgentCore-runtime labels Aug 25, 2025
@neylsoncrepalde neylsoncrepalde force-pushed the byo-container-streaming-example branch from 56860f7 to 009cbb3 Compare September 2, 2025 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

01-AgentCore-runtime 01-tutorials/01-AgentCore-runtime 01-tutorials 01-tutorials

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants