Skip to content

feat(bedrock): Set gen_ai.request.model attribute - #8231

Draft
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/bedrock/gen-ai-response-model
Draft

alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/bedrock/gen-ai-response-model

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Collaborator

Set the requested model based on the modelId parameter to the Bedrock API.

@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Coverage Results 📊

✅ 137254 passed | ❌ 1 failed | ⏭️ 7575 skipped | Total: 144830 | Pass Rate: 94.77% | Execution Time: 432m 31s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -28
Passed Tests 📉 -29
Failed Tests 📈 +1
Skipped Tests —

➕ New Tests (1)

View new tests
  • test_key_as_string[key7-]
    • File: tests.integrations.redis.test_redis_cache_module
    • Status: ❌ Failing

❌ Failed Tests

test_key_as_string[key7-]

File: tests.integrations.redis.test_redis_cache_module
Suite: py3.9-redis-v6.4.0
Error: AssertionError: assert ';݇hhEN\u1754\x0bUk +' == ''
 + ;݇hhEN᝔#x0B
 + Uk +

Stack Trace
tests/integrations/redis/test_redis_cache_module.py:529: in test_key_as_string
    assert _key_as_string(key) == expected_key
E   AssertionError: assert ';݇hhEN\u1754\x0bU`k +' == ''
E     + ;݇hhEN᝔#x0B
E     + U`k +

✅ Patch coverage is 100.00% (5 of 5 changed executable lines covered; target 80%).
Project statement coverage is 90.31% (up 0.02 percentage points from base (da37f0a) to head (c747d65)).

Changed files with executable lines (1)
File Patch coverage Changed executable lines
sentry_sdk/integrations/boto3/_services/bedrock.py 100.00% 5/5 covered; partial branches: 38
Coverage diff
@@            Coverage Diff             @@
##        master     #8231       +/-##
==========================================
+ Coverage    90.29%    90.31%    +0.02%
==========================================
  Files          203       203         —
  Tracked lines     26953     26957        +4
  Branches     10066     10068        +2
==========================================
+ Hits         24338     24346        +8
+ Misses        2615      2611        -4
- Partials      1520      1524        +4

Generated by Coverage Action

This branch has not been deployed

No deployments
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