-
Notifications
You must be signed in to change notification settings - Fork 64
docs: address feedback and improve Runtime/Gateway documentation #163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…d explicit region configuration guidance - Clarify ARM64 requirements and rationale - Restructure Gateway quickstart with numbered steps - Add Console resource links for better navigation - Document uv as recommended package manager - Create troubleshooting guide for common issues - Improve CDK integration documentation
|
||
The `agentcore` CLI provides commands for configuring, launching, managing agents, and working with gateways. | ||
|
||
## Recent Improvements |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is duplicate I think, commit log would already give this info
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i am good removing this - thought we should let the users know about the recent improvement we have made in the docs
|
||
- `--user-id, -u TEXT`: User ID for authorization flows | ||
|
||
**Enhanced Output:** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets just call it example outputs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
@@ -1,528 +1,247 @@ | |||
# QuickStart: A Fully Managed MCP Server in 5 Minutes! 🚀 | |||
|
|||
Amazon Bedrock AgentCore Gateway provides an easy and secure way for developers to build, deploy, discover, and connect to tools at scale. AI agents need tools to perform real-world tasks—from querying databases to sending messages to analyzing documents. With Gateway, developers can convert APIs, Lambda functions, and existing services into Model Context Protocol (MCP)-compatible tools and make them available to agents through Gateway endpoints with just a few lines of code. Gateway supports OpenAPI, Smithy, and Lambda as input types, and is the only solution that provides both comprehensive ingress authentication and egress authentication in a fully-managed service. Gateway eliminates weeks of custom code development, infrastructure provisioning, and security implementation so developers can focus on building innovative agent applications. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was specifically added by Nick, lets confirm before changing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have not changed the content - just converted them into bullet points based on workshop feedback
} | ||
``` | ||
|
||
### What Gets Auto-Created |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should keep this what get auto created
- **AWS Account** with credentials configured (`aws configure`) | ||
- **AWS Account** with credentials configured in your desired region (`aws configure`) | ||
- **Python 3.10+** installed | ||
- **AWS Permissions**: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is important, lets not remove it
curl -LsSf https://astral.sh/uv/install.sh | sh | ||
|
||
# Clone and install | ||
git clone https://github.com/aws/bedrock-agentcore-starter-toolkit.git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why git clone?
⚠️ **Important**: AgentCore Runtime requires ARM64 architecture containers. This is because: | ||
- Runtime uses AWS Graviton processors for cost-efficiency and performance | ||
- All containers must be built for `linux/arm64` platform | ||
- Local development on x86 machines requires Docker buildx or CodeBuild deployment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets clarify launch uses code build by default, and move docker buildx to after
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
cat > requirements.txt << EOF | ||
bedrock-agentcore | ||
strands-agents | ||
EOF |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when does user add these?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed this step.
README.md
Outdated
|
||
```bash | ||
# Clone the repository | ||
git clone https://github.com/aws/bedrock-agentcore-starter-toolkit.git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why git clone?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed this.
README.md
Outdated
pip install -e . | ||
``` | ||
|
||
### About Package Management |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this move to contributions.md?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved
### Step 3: Deploy and Test | ||
|
||
```bash | ||
# Deploy the agent with Gateway integration | ||
agentcore configure --entrypoint weather_agent.py | ||
agentcore launch | ||
|
||
# Test the integration | ||
agentcore invoke '{"prompt": "What is the weather in Seattle?"}' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do i now need to do this via CLI..? Ideally I should not.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed the CLI - it was a mistake. please check again
```bash | ||
# List resources | ||
agentcore status | ||
|
||
# Remove specific agent | ||
agentcore destroy --agent weather-agent | ||
|
||
# Clean up Gateway (via console or SDK) | ||
# Note: Cognito resources may need manual cleanup |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this doing? we should just include this in the python.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed this section
|
||
## Best Practices | ||
|
||
1. **Same Region**: Deploy Gateway and Runtime in the same region |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are we talking about Runtime here? Remove this please
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed references to Runtime
# Deploy the agent with Gateway integration | ||
agentcore configure --entrypoint weather_agent.py | ||
agentcore launch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this please. We should not include anything about AgentCore Runtime in this guide.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed references to Runtime
- Completed [Runtime QuickStart](../runtime/quickstart.md) | ||
- Agent deployed and running | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed references to Runtime
Amazon Bedrock AgentCore Gateway provides an easy and secure way for developers to build, deploy, discover, and connect to tools at scale. AI agents need tools to perform real-world tasks—from querying databases to sending messages to analyzing documents. With Gateway, developers can convert APIs, Lambda functions, and existing services into Model Context Protocol (MCP)-compatible tools and make them available to agents through Gateway endpoints with just a few lines of code. Gateway supports OpenAPI, Smithy, and Lambda as input types, and is the only solution that provides both comprehensive ingress authentication and egress authentication in a fully-managed service. Gateway eliminates weeks of custom code development, infrastructure provisioning, and security implementation so developers can focus on building innovative agent applications. | ||
Amazon Bedrock AgentCore Gateway provides an easy and secure way for developers to build, deploy, discover, and connect to tools at scale. | ||
|
||
In the quick start guide you will learn how to set up a Gateway and integrate it into your agents using the AgentCore Starter Toolkit. You can find more comprehensive guides and examples [**here**](https://github.com/awslabs/amazon-bedrock-agentcore-samples/tree/main/01-tutorials/02-AgentCore-gateway). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why are we removing this?
bedrock-agentcore | ||
strands-agents | ||
EOF | ||
echo "strands-agents" > requirements.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bedrock-agentcore should also be here
|
||
```bash | ||
# Configure and deploy (auto-creates all required resources) | ||
agentcore configure -e my_agent.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets use -e
### LangGraph Example | ||
|
||
### Getting Help | ||
Create `langgraph_agent.py`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets move this to a separate path, maybe in examples
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
created a new file under examples and moved both Langgraph and CrewAI there. Although we should advertise somewhere that we work seamlessly with other frameworks too - improves discoverability for this repo
README.md
Outdated
pip install bedrock-agentcore-starter-toolkit | ||
``` | ||
|
||
### Development Setup |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should this got to contributions.md?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
mkdir agentcore-runtime-quickstart | ||
cd agentcore-runtime-quickstart | ||
python3 -m venv venv | ||
source venv/bin/activate | ||
uv venv | ||
source .venv/bin/activate # On Windows: .venv\Scripts\activate | ||
|
||
# Install packages | ||
uv pip install bedrock-agentcore-starter-toolkit strands-agents |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this can be simplified to:
- uv init agentcore-runtime-quickstart && cd agentcore-runtime-quickstart
- uv add bedrock-agentcore strands-agents
- source .venv/bin/activate
agentecore configure -e main.py
agentcore launch
agentcore invoke "tell me a joke"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think quick start should have a single path to start with, lets continue without uv and capture uv after initial e2e. and keep it very brief.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved uv to advanced setup section.. removed the clutter in the Setup section
# Setup project | ||
mkdir agentcore-runtime-quickstart | ||
cd agentcore-runtime-quickstart | ||
python3 -m venv venv |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lets use .venv
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
…eadMe, added comprehensive Gateway examples
Response: | ||
Your formatted response here | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question: do the Gateway CLI sections below this work?
# IMPORTANT: Set your AWS region here | ||
REGION = "us-west-2" # Change to your preferred region | ||
|
||
def fix_gateway_role_trust_policy(role_name, account_id, region): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion: please make this part of the agentcore-starter-toolkit if it's needed. This should not be in quickstart guide.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
```bash | ||
# Delete Gateway and all resources | ||
aws bedrock-agentcore-control delete-gateway \ | ||
--gateway-identifier YOUR_GATEWAY_ID \ | ||
--region us-east-1 | ||
|
||
# Delete Cognito pool (optional) | ||
aws cognito-idp delete-user-pool \ | ||
--user-pool-id YOUR_POOL_ID \ | ||
--region us-east-1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we just do this as part of python as well given rest of guide is in python?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
def test_mcp_request(method, params=None): | ||
"""Test an MCP request""" | ||
headers = { | ||
"Authorization": f"Bearer {config['access_token']}", | ||
"Content-Type": "application/json" | ||
} | ||
|
||
payload = { | ||
"jsonrpc": "2.0", | ||
"id": str(int(time.time())), | ||
"method": method, | ||
"params": params or {} | ||
} | ||
|
||
try: | ||
response = requests.post( | ||
config['gateway_url'], | ||
headers=headers, | ||
json=payload, | ||
timeout=10 | ||
) | ||
|
||
print(f" Status: {response.status_code}") | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue: we should not be hand-crafting MCP requests. Ideally we should rely on SDK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was an optional diagnostic code that i had added.. i have removed this and updated the troubleshooting section
# ============= TEST 2: Verify Gateway Role ============= | ||
print("\\n" + "=" * 60) | ||
print("TEST 2: Gateway Execution Role") | ||
print("-" * 60) | ||
|
||
role_arn = config.get('gateway_role_arn', f"arn:aws:iam::{account_id}:role/AgentCoreGatewayExecutionRole") | ||
role_name = role_arn.split('/')[-1] | ||
|
||
try: | ||
role = iam.get_role(RoleName=role_name) | ||
print(f"✅ Role exists: {role_arn}") | ||
|
||
# Check trust policy | ||
trust_policy = role['Role']['AssumeRolePolicyDocument'] | ||
can_be_assumed = any( | ||
stmt.get('Principal', {}).get('Service') == 'bedrock-agentcore.amazonaws.com' | ||
for stmt in trust_policy['Statement'] | ||
) | ||
|
||
if can_be_assumed: | ||
print("✅ Role can be assumed by bedrock-agentcore") | ||
else: | ||
print("❌ Role cannot be assumed by bedrock-agentcore") | ||
|
||
# Check attached policies | ||
attached = iam.list_attached_role_policies(RoleName=role_name) | ||
print(f"📋 Attached policies: {len(attached['AttachedPolicies'])}") | ||
for policy in attached['AttachedPolicies']: | ||
print(f" • {policy['PolicyName']}") | ||
|
||
# Check inline policies | ||
inline = iam.list_role_policies(RoleName=role_name) | ||
print(f"📋 Inline policies: {len(inline['PolicyNames'])}") | ||
for policy_name in inline['PolicyNames']: | ||
print(f" • {policy_name}") | ||
|
||
# Get policy details | ||
policy_doc = iam.get_role_policy(RoleName=role_name, PolicyName=policy_name) | ||
statements = policy_doc['PolicyDocument']['Statement'] | ||
|
||
# Check for Lambda invoke permission | ||
has_lambda_invoke = any( | ||
'lambda:InvokeFunction' in stmt.get('Action', []) or | ||
'lambda:*' in stmt.get('Action', []) | ||
for stmt in statements | ||
) | ||
|
||
if has_lambda_invoke: | ||
print(" ✅ Has lambda:InvokeFunction permission") | ||
else: | ||
print(" ❌ Missing lambda:InvokeFunction permission") | ||
|
||
except Exception as e: | ||
print(f"❌ Role error: {e}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all this stuff should be in the toolkit itself!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added setup_gateway.py
for Gateway creation, OAuth setup, and testing... updated the quickstart documentation
### AWS PrivateLink Support | ||
|
||
Gateway now supports private VPC connectivity: | ||
|
||
```python | ||
# Create VPC endpoint for private access | ||
# Service name: com.amazonaws.{region}.bedrock-agentcore.gateway | ||
|
||
# Example endpoint policy for OAuth-based Gateway | ||
vpc_endpoint_policy = { | ||
"Statement": [{ | ||
"Principal": "*", # Required for OAuth (non-SigV4) authentication | ||
"Effect": "Allow", | ||
"Action": "*", | ||
"Resource": f"arn:aws:bedrock-agentcore:{REGION}::gateway/{gateway_id}" | ||
}] | ||
} | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this in the quickstart? This seems like info that would come in advanced topics in the very end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point - i can move this to the Advanced topic section
print(f"⚠️ Warning: Could not update role trust policy: {e}") | ||
return False | ||
|
||
def main(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
way way too much code here. Can we please figure out how to put code in the starter toolkit itself as needed so that the quickstart remains just a few lines of code?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved these into toolkit
…umentation Add setup_gateway.py to simplify gateway creation to just 3 lines of code. Update quickstart.md with express setup path for new users. Add unit tests for the new functionality.
class QuickGateway: | ||
"""One-line Gateway setup for quickstart""" | ||
|
||
def __init__(self, region="us-east-1"): | ||
self.region = region | ||
self.client = GatewayClient(region_name=region) | ||
self.config_file = Path("gateway_config.json") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why are we creating a QuickGateway on top of Gateway which is already supposed to be quick start? can we just incorporate any improvements (like IAM permission fixes) in other one. Also please do not club together creating authorizer, gateway, and target. It makes things very amibguous to the user what is actually going on behind the scenes and makes it very hard to graduate to more sophisticated usage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed - removed QuickGateway entirely. IAM fixes and all improvements now live directly in GatewayClient as methods. Users call individual steps explicitly.
# Import here to avoid circular dependencies | ||
from strands import Agent | ||
from strands.models import BedrockModel | ||
from strands.tools.mcp.mcp_client import MCPClient | ||
from mcp.client.streamable_http import streamablehttp_client |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure what circular deps this refers to -- we should import like normal. Also my suggestion would be to keep the non-gateway agent interactions in the quickstart (what we have today) so it is clear how gateway is used with an agent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed - no circular deps existed anyway. Agent code now stays entirely in quickstart as example usage, not in the gateway library.
def cleanup(self): | ||
"""Remove all resources""" | ||
if not self.config_file.exists(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add this to the normal gateway client code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done - cleanup_gateway()
is now a method in GatewayClient, not a separate class.
except Exception as e: | ||
print(f" ⚠️ IAM role update failed: {str(e)}. Continuing with best effort.") | ||
|
||
def test(self): # pragma: no cover |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why repeat this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed - QuickGateway is gone, so no more duplicate code or coverage issues.
# Create and test Gateway | ||
gateway = QuickGateway(region="us-west-2") # enter your preferred region | ||
gateway.create() # Creates Gateway, Lambda tools, and OAuth | ||
gateway.test() # Interactive agent chat | ||
# gateway.cleanup() # Run this when done to remove resources |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this completely defeats the purpose of actually teaching the user what is going on. that is why we have the clear code that was deleted showing the steps to create authorizer, gateway, and target.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yea, which is why my original version had each step written in the quickstart to educate the users but it looked like wanted it moved to the toolkit. we have to choose - whether to keep the code in quickstart vs toolkit. #163 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Completely restructured - quickstart now shows each step explicitly (create authorizer → create gateway → fix IAM → add target). Users see and control every operation.
|
||
<details> | ||
<summary>Advanced: AWS PrivateLink for VPC Connectivity</summary> | ||
|
||
Create private connection between your VPC and Gateway: | ||
|
||
```bash | ||
aws ec2 create-vpc-endpoint \ | ||
--vpc-id vpc-12345678 \ | ||
--service-name com.amazonaws.region.bedrock-agentcore.gateway | ||
``` | ||
|
||
</details> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this even belong in quickstart? Maybe at bottom of advanced section? also why we adding another details section instead details section?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
its a small section where they can learn to add VPC if they chose to
</details> | ||
|
||
|
||
## Learn More |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No one will read to bottom to see these. These should be at top.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved it up - there are links to samples, boto3 doc and developer guide anyway in the first 2 paragraphs.
action = event.get('action') | ||
|
||
# Your business logic here | ||
result = process_customer_data(customer_id, action) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this code even work? We should only have code that actually functions here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was an existing code that wanted to showcase custom lambda tools. let me know if you want it removed for now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed non-working placeholder code. All examples now tested and functional.
Here is an example of a simple Lambda function that implements these tools. | ||
```python | ||
import json | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why are we removing this...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Preserved - the working Lambda handler example is back in the quickstart, now showing actual implementation of get_weather and get_time tools
### Invocation Logging | ||
|
||
Monitor all Gateway invocations with: | ||
|
||
- **CloudWatch Logs**: Real-time analysis at `/aws/bedrock-agentcore/gateways/{gateway-id}` | ||
- **Amazon S3**: Long-term storage | ||
- **Amazon Data Firehose**: Stream to analytics services | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no idea what this means... we have not even shown how to set this up!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed the feature announcement
…w to: Create a Strands agent with short-term and long-term memory Add Gateway integration with a calculator tool Deploy the agent to AgentCore Runtime Test memory persistence and tool usage
Signed-off-by: Sundar Raghavan <[email protected]>
Add explicit region configuration guidance
Type of Change
Testing
Checklist
Security Checklist
Breaking Changes
List any breaking changes and migration instructions:
N/A
Additional Notes
Add any additional notes or context about the PR here.