Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add system and provider attributes to openai spans #1075

Merged
merged 3 commits into from
Oct 21, 2024

Conversation

cephalization
Copy link
Contributor

Additionally export LLM_SYSTEM and LLM_PROVIDER attribute keys within SemanticConventions

Additionally export LLM_SYSTEM and LLM_PROVIDER attribute keys within SemanticConventions
@cephalization cephalization requested a review from a team as a code owner October 21, 2024 18:20
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Oct 21, 2024
Copy link
Contributor

github-actions bot commented Oct 21, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@cephalization
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@cephalization
Copy link
Contributor Author

recheck

github-actions bot added a commit that referenced this pull request Oct 21, 2024
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Oct 21, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 21, 2024
@cephalization cephalization merged commit f965410 into main Oct 21, 2024
3 checks passed
@cephalization cephalization deleted the cephalization/5102-llm-system-span branch October 21, 2024 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants