Skip to content

fix: cleanup scripts skip DEFAULT endpoint to prevent resource leaks - #2007

Open
jyotsnamas wants to merge 1 commit into
awslabs:mainfrom
jyotsnamas:fix/cleanup-default-endpoint
Open

fix: cleanup scripts skip DEFAULT endpoint to prevent resource leaks#2007
jyotsnamas wants to merge 1 commit into
awslabs:mainfrom
jyotsnamas:fix/cleanup-default-endpoint

Conversation

@jyotsnamas

Copy link
Copy Markdown
Contributor

Summary

  • Fix 18 cleanup.py scripts that silently leak AgentCore runtimes and endpoints
  • The API auto-manages DEFAULT endpoints — they cannot be explicitly deleted
  • Scripts that attempt to delete DEFAULT get ConflictException, which then prevents runtime deletion
  • Scripts report "Cleanup complete" while resources remain alive and accruing costs

Root cause

1. cleanup.py calls DeleteAgentRuntimeEndpoint(endpointName="DEFAULT")
   → ConflictException: "Default endpoints are removed when you delete the agent"
2. cleanup.py calls DeleteAgentRuntime
   → ConflictException: "This agent has endpoints that must be deleted first"
3. Both exceptions caught silently → "Cleanup complete" printed
4. Runtime + endpoint remain READY in AWS

Fix

Skip DEFAULT endpoints during explicit deletion, matching the pattern already used in 01-strands-bedrock/cleanup.py:

for ep in endpoints.get("runtimeEndpoints", []):
    if ep["name"] == "DEFAULT":
        continue
    control.delete_agent_runtime_endpoint(...)

Verification

Deployed and cleaned up multiple modules in us-west-2:

  • Without fix: aws bedrock-agentcore-control list-agent-runtimes shows runtime still READY after cleanup
  • With fix: Runtime successfully deleted, confirmed gone via list-agent-runtimes

Test plan

  • Deploy 02-a2a-protocol → cleanup now deletes runtime successfully
  • Deploy 09-mcp-e2e/01-server-e2e → confirmed leaked runtime with old cleanup, deleted with direct API call
  • Verified strands-bedrock cleanup (already has the fix) works correctly

🤖 Generated with Claude Code

The AgentCore API now auto-manages DEFAULT endpoints — they cannot be
explicitly deleted. cleanup.py scripts that attempt to delete all
endpoints (including DEFAULT) get a ConflictException, which then
prevents the runtime from being deleted. Resources are leaked despite
the script reporting "Cleanup complete".

Fix: Skip DEFAULT endpoints during explicit deletion, matching the
pattern already used in 01-strands-bedrock/cleanup.py. Also fixes
the observability cleanup which used wrong parameter name (name=
instead of endpointName=).

Verified by live deployment testing: deploy → invoke → cleanup now
succeeds without leaking runtimes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant