Skip to content

feat(openai_agents): Gate tool execution span data on data_collection options - #7123

Open
ericapisani wants to merge 2 commits into
masterfrom
py-2588-openai-agents-execute-tool
Open

feat(openai_agents): Gate tool execution span data on data_collection options#7123
ericapisani wants to merge 2 commits into
masterfrom
py-2588-openai-agents-execute-tool

Conversation

@ericapisani

@ericapisani ericapisani commented Aug 6, 2026

Copy link
Copy Markdown
Member

Respect the data_collection configuration for gen_ai.tool.input and
gen_ai.tool.output attributes in execute_tool spans, falling back to
send_default_pii for backwards compatibility. This brings tool execution
spans in line with other gen_ai span data collection gates.

Refs PY-2588

… options

Respect the data_collection configuration for gen_ai.tool.input and
gen_ai.tool.output attributes in execute_tool spans, falling back to
send_default_pii for backwards compatibility. This brings tool execution
spans in line with other gen_ai span data collection gates.

Refs PY-2588
@linear-code

linear-code Bot commented Aug 6, 2026

Copy link
Copy Markdown

PY-2588

@ericapisani
ericapisani marked this pull request as ready for review August 7, 2026 14:02
@ericapisani
ericapisani requested a review from a team as a code owner August 7, 2026 14:02
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

99559 passed | ⏭️ 6485 skipped | Total: 106044 | Pass Rate: 93.88% | Execution Time: 358m 41s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +300
Passed Tests 📈 +294
Failed Tests
Skipped Tests 📈 +6

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.99%    90.00%    +0.01%
==========================================
  Files          193       193         —
  Lines        24991     25006       +15
  Branches      9012      9024       +12
==========================================
+ Hits         22490     22506       +16
- Misses        2501      2500        -1
- Partials      1437      1436        -1

Generated by Codecov Action

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