Skip to content

docs(bedrock): add Converse API permissions and one-click deployment link#11093

Closed
JiaDe-Wu wants to merge 1 commit into
NousResearch:mainfrom
JiaDe-Wu:docs/bedrock-quickstart
Closed

docs(bedrock): add Converse API permissions and one-click deployment link#11093
JiaDe-Wu wants to merge 1 commit into
NousResearch:mainfrom
JiaDe-Wu:docs/bedrock-quickstart

Conversation

@JiaDe-Wu

@JiaDe-Wu JiaDe-Wu commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

Follow-up fixes for the Bedrock provider docs merged in #10549.

Fixes

  1. aws-bedrock.md — IAM permissions were jammed into one backtick string with an extra parenthesis. Split into proper list items: Converse API permissions + Anthropic SDK permissions.
  2. aws-bedrock.md — One-Click Deployment section had literal backslash-n instead of newlines (was appended via echo). Replaced with proper markdown.
  3. quickstart.md — Bedrock was missing from the provider table. Added row with link to the Bedrock guide.
  4. config.py — Bedrock was missing from the fallback_model Supported providers comment (2 occurrences).

3 files, +9 lines.

@JiaDe-Wu

Copy link
Copy Markdown
Contributor Author

@teknium1 Tiny docs follow-up to #10549 — just two lines. Adds the Converse API IAM permissions that were missing from the prerequisites, and a link to the CloudFormation deployment template. Should be safe to merge as-is.

@JiaDe-Wu JiaDe-Wu force-pushed the docs/bedrock-quickstart branch from 4870a07 to 6d971ba Compare April 16, 2026 15:45
@JiaDe-Wu

Copy link
Copy Markdown
Contributor Author

Updated — expanded this from 2 lines to 4 fixes across 3 files:

  1. IAM permissions in aws-bedrock.md were broken (extra parenthesis, all jammed in one backtick)
  2. One-Click Deployment section had literal backslash-n instead of newlines
  3. Bedrock missing from quickstart.md provider table
  4. Bedrock missing from config.py fallback provider comments

@teknium1 all follow-up to #10549, 9 lines total.

@JiaDe-Wu

Copy link
Copy Markdown
Contributor Author

CI status note — all 3 failures are pre-existing, unrelated to this docs-only PR:

  • test: 1 flaky test (test_preflight_compresses_oversized_history) — 12103 passed, 1 failed. Not a file we touched.
  • docs-site-checks: 11 errors in configuration.md (not modified in this PR).
  • check-attribution: contributor email mapping — needs maintainer .mailmap update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have provider/bedrock AWS Bedrock (boto3, IAM) type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants