Skip to content

ref(openai-agents): Stop setting gen_ai.request.available_tools on Execute Tool spans - #6954

Open
alexander-alderman-webb wants to merge 4 commits into
webb/openai-responses-gen-ai-tool-definitionsfrom
webb/openai-agents-tool-definitions
Open

ref(openai-agents): Stop setting gen_ai.request.available_tools on Execute Tool spans#6954
alexander-alderman-webb wants to merge 4 commits into
webb/openai-responses-gen-ai-tool-definitionsfrom
webb/openai-agents-tool-definitions

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Description

This doesn't make any sense. A tool execution doesn't have available tools.

Issues

Reminders

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

99633 passed | ❌ 1 failed | ⏭️ 6757 skipped | Total: 106391 | Pass Rate: 93.65% | Execution Time: 358m 29s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -160
Passed Tests 📉 -136
Failed Tests
Skipped Tests 📉 -24

➕ New Tests (1)

View new tests
  • test_continuous_profiler_auto_start_and_stop_sampled[non-experiment-gevent]
    • File: tests.profiler.test_continuous_profiler
    • Status: ❌ Failing

➖ Removed Tests (1)

View removed tests
  • test_continuous_profiler_auto_start_and_stop_sampled_span_streaming[non-experiment-thread]
    • File: tests.profiler.test_continuous_profiler

❌ Failed Tests

test_continuous_profiler_auto_start_and_stop_sampled[non-experiment-gevent]

File: tests.profiler.test_continuous_profiler
Suite: py3.6-gevent
Error: AssertionError: profiler should not be running assert '2bcfa090f5db41139dc441407421b6e2' is None + where '2bcfa090f5db41139dc441407421b6e2' = get_profiler_id()

Stack Trace
tests/profiler/test_continuous_profiler.py:812: in test_continuous_profiler_auto_start_and_stop_sampled
    assert get_profiler_id() is None, "profiler should not be running"
E   AssertionError: profiler should not be running
E   assert '2bcfa090f5db41139dc441407421b6e2' is None
E    +  where '2bcfa090f5db41139dc441407421b6e2' = get_profiler_id()

✅ Patch coverage is 100.00%. Project has 2504 uncovered lines.
✅ Project coverage is 89.89%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.89%    89.89%        —%
==========================================
  Files          193       193         —
  Lines        24686     24756       +70
  Branches      8814      8862       +48
==========================================
+ Hits         22188     22252       +64
- Misses        2498      2504        +6
- Partials      1400      1415       +15

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review July 31, 2026 08:55
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner July 31, 2026 08:55
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