newrelic / newrelic-python-agent

New Relic Python Agent
https://docs.newrelic.com/docs/agents/python-agent
Apache License 2.0
176 stars 99 forks source link

Remove transaction ID attribute #1090

Closed lrafeei closed 6 months ago

lrafeei commented 6 months ago

Overview

This PR removes transaction_id as an LLM attribute.

Wait to merge until the transaction_id -> trace_id transition is complete.

github-actions[bot] commented 6 months ago

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON bandit 3 0 6.45s
✅ PYTHON black 20 0 0 2.98s
✅ PYTHON flake8 20 0 1.69s
✅ PYTHON isort 20 0 0 0.25s
✅ PYTHON pylint 20 0 10.63s

See detailed report in MegaLinter reports _Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff_

_MegaLinter is graciously provided by OX Security_

codecov-commenter commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.10%. Comparing base (b3489bf) to head (f9a55bb). Report is 1 commits behind head on develop-ai-limited-preview-3.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop-ai-limited-preview-3 #1090 +/- ## ================================================================ - Coverage 81.86% 80.10% -1.76% ================================================================ Files 192 192 Lines 21252 21252 Branches 3711 3711 ================================================================ - Hits 17397 17023 -374 - Misses 2782 3183 +401 + Partials 1073 1046 -27 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.