feat: add Bedrock Managed Knowledge Base support across samples - #285
feat: add Bedrock Managed Knowledge Base support across samples#285PVidyadhar wants to merge 1 commit into
Conversation
1708b3c to
60f275c
Compare
|
Latest scan for commit: ✅ Security Scan Report (PR Files Only)Scanned Files
Security Scan Results
Threshold: High No security issues detected in your changes. Great job! This scan only covers files changed in this PR. |
087f0cb to
98230ae
Compare
- Finance advisor: KB creation supports MANAGED type (default: VECTOR), retrieval uses correct config - Lambda error analysis: CDK stack creates MANAGED KB when knowledgeBaseType=MANAGED context provided - Agentic RAG tool: supports managedSearchConfiguration + AgenticRetrieveStream - Learn tutorial: KB setup prereqs support MANAGED type - Updated README with managed KB section and doc links - Added BEDROCK_MANAGED_KB.md design doc - Existing VECTOR samples unchanged
98230ae to
9a69969
Compare
|
@PVidyadhar Hi, could you please test all the changes to make sure its working? For example, the retrieve tool doesn't actually pick up the MANAGED setting. Even with KNOWLEDGE_BASE_TYPE=MANAGED, the tool always sends vectorSearchConfiguration, and Bedrock rejects that against a managed KB. |
@manoj-selvakumar5 Thanks for catching this! You're right — strands_tools.retrieve doesn't support KNOWLEDGE_BASE_TYPE=MANAGED and always sends The Strands team closed our PR to the tools repo (#534) and recommends using the built-in memory solution instead For the samples that use strands_tools.retrieve (lambda error analysis + finance advisor), I can fix them by using direct boto3 with Would that approach work, or would you prefer these samples migrate to the built-in memory pattern? |
Issue #, if available:
N/A — new feature addition for AWS Bedrock Managed Knowledge Base GA launch.
Description of changes:
Added Bedrock Managed Knowledge Base support across multiple samples in the repository.
Changes:
knowledgeBaseType=MANAGEDcontext providedmanagedSearchConfiguration+ AgenticRetrieveStreamTesting:
--kb-type MANAGEDBy submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.