Skip to content

feat(bedrock): Record inferenceConfig parameters - #8232

Merged
alexander-alderman-webb merged 3 commits into
masterfrom
webb/bedrock/request-parameters
Nov 10, 2026
Merged

alexander-alderman-webb merged 3 commits into
masterfrom
webb/bedrock/request-parameters

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Collaborator

Set the maximum output tokens, temperature and top p request parameters from the Bedrock API.

@alexander-alderman-webb
alexander-alderman-webb force-pushed the webb/bedrock/request-parameters branch from e18dff0 to 9c7ab68 Compare November 9, 2026 14:25
@alexander-alderman-webb alexander-alderman-webb changed the title feat(bedrock): Record request parameters feat(bedrock): Record inferenceConfig parameters Nov 9, 2026
@github-actions

github-actions Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Coverage Results 📊

✅ 137332 passed | ⏭️ 7575 skipped | Total: 144907 | Pass Rate: 94.77% | Execution Time: 423m 22s

All tests are passing successfully.

✅ Patch coverage is 91.67% (11 of 12 changed executable lines covered; target 80%).
Project statement coverage is 90.29% (down 0.02 percentage points from base (f73ce4e) to head (86b5dbd)).

Changed files with executable lines (1)
File Patch coverage Changed executable lines
sentry_sdk/integrations/boto3/_services/bedrock.py 91.67% 11/12 covered; missed: 44; partial branches: 43, 47, 51, 55
Coverage diff
@@            Coverage Diff             @@
##        master     #8232       +/-##
==========================================
- Coverage    90.31%    90.29%    -0.02%
==========================================
  Files          203       203         —
  Tracked lines     26964     26980       +16
  Branches     10070     10080       +10
==========================================
+ Hits         24351     24360        +9
- Misses        2613      2620        +7
- Partials      1526      1529        +3

Generated by Coverage Action

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review November 10, 2026 11:35
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner November 10, 2026 11:35
@alexander-alderman-webb
alexander-alderman-webb force-pushed the webb/bedrock/request-parameters branch from add357c to 86b5dbd Compare November 10, 2026 14:35
@alexander-alderman-webb
alexander-alderman-webb enabled auto-merge (squash) November 10, 2026 14:47
@alexander-alderman-webb
alexander-alderman-webb merged commit 85232d7 into master Nov 10, 2026
171 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/bedrock/request-parameters branch November 10, 2026 14:48
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.

2 participants