Skip to content

chore(openai-agents): Only set gen_ai.provider.name on AI Client spans - #7691

Merged
alexander-alderman-webb merged 2 commits into
major/3.0from
webb/openai-agents/limit-provider-name
Oct 5, 2026
Merged

alexander-alderman-webb merged 2 commits into
major/3.0from
webb/openai-agents/limit-provider-name

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Sep 23, 2026 •

Copy link
Copy Markdown
Collaborator

Description

See explanation in open-telemetry/semantic-conventions-genai#289

The gen_ai.provider.name indicates which provider a model request is made to, and has no meaning for agent invocations.

Issues

Reminders

@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ 57096 passed | ❌ 2 failed | ⏭️ 2727 skipped | Total: 59825 | Pass Rate: 95.44% | Execution Time: 154m 59s

❌ Failed Tests

test_cache_spans_decorator

File: tests.integrations.django.test_cache_module
Suite: py3.11-django-v4.2.30
Error: tests/integrations/django/test_cache_module.py:307: in test_cache_spans_decorator assert not spans[0]["attributes"]["cache.hit"] E assert not True

Stack Trace
tests/integrations/django/test_cache_module.py:307: in test_cache_spans_decorator
    assert not spans[0]["attributes"]["cache.hit"]
E   assert not True

test_db_transaction_spans_disabled_no_autocommit

File: tests.integrations.django.test_db_transactions
Suite: py3.12-django-v4.2.30
Error: tests/integrations/django/test_db_transactions.py:136: in test_db_transaction_spans_disabled_no_autocommit assert postgres_commit["is_segment"] is True E assert False is True

Stack Trace
tests/integrations/django/test_db_transactions.py:136: in test_db_transaction_spans_disabled_no_autocommit
    assert postgres_commit["is_segment"] is True
E   assert False is True

✅ Patch coverage is 100.00%. Project has 2078 uncovered lines.
❌ Project coverage is 90.03%. Comparing base (8c2d49e) to head (3e35d63).

Coverage diff
@@            Coverage Diff             @@
##        master       #PR       +/-##
==========================================
- Coverage    90.34%    90.03%    -0.31%
==========================================
  Files          202       185       -17
  Lines        26551     20852     -5699
  Branches      9856      7238     -2618
==========================================
+ Hits         23988     18774     -5214
- Misses        2563      2078      -485
- Partials      1487      1207      -280

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review October 5, 2026 07:03
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner October 5, 2026 07:03
@alexander-alderman-webb
alexander-alderman-webb merged commit b565fbd into major/3.0 Oct 5, 2026
143 of 145 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/openai-agents/limit-provider-name branch October 5, 2026 08:34
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