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.

Sign up for GitHub

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

Jump to bottom

Release Exporter support for grpc protocol using OTEL_EXPORTER_OTLP_PROTOCOL #439

Merged
merged 30 commits into from
Dec 31, 2024
Merged

Release Exporter support for grpc protocol using OTEL_EXPORTER_OTLP_PROTOCOL #439

merged 30 commits into from
Dec 31, 2024

Conversation

alizenhom
Copy link
Collaborator

Description

Please include a summary of the changes and the related issue to help is review the PR better and faster.

Checklist for adding new integration:

  • Defined APIS in constants folder.
  • Updated SERVICE_PROVIDERS in common.py
  • Created a folder under instrumentation with the name of the integration with atleast patch.py and instrumentation.py files.
  • Added instrumentation in all_instrumentations in langtrace.py and to the InstrumentationType in types.py files.
  • Added examples for the new integration in the examples folder.
  • Updated pyproject.toml to install new dependencies
  • Updated the README.md of langtrace-python-sdk to include the new integration in the supported integrations table.
  • Updated the README.md of Langtrace's repository to include the new integration in the supported integrations table.
  • Added new integration page to supported integrations in Langtrace Docs

obinnascale3 and others added 30 commits December 2, 2024 20:34
@obinnascale3
Ensure non-null check for attribute
3eb0dcd
@devin-ai-integration
fix: Make base_url handling consistent and improve Azure test
5306f85
@devin-ai-integration
test: Improve Azure embeddings test with mocking
18e50b5
@devin-ai-integration
test: Improve Azure embeddings test with better HTTP mocking
ff4c177
@devin-ai-integration
test: Update Azure test URL assertion to match client behavior
9b39ba4
@devin-ai-integration
fix: Make base_url handling consistent in embeddings functions
5a056b2
@devin-ai-integration
chore: Add debug logging for provider detection
92b7d89
@devin-ai-integration
chore: Add debug logging to get_base_url
768d035
@devin-ai-integration
chore: Add more debug logging for Azure provider detection
e705b4d
@devin-ai-integration
test: Initialize OpenAIInstrumentation in test setup
88d6883
@devin-ai-integration
test: Fix SpanAttributes import path
ffd3f31
@devin-ai-integration
test: Add test cassette for embeddings base URL test
0b03539
@devin-ai-integration
chore: Remove debug print statements
d8a48f0
@devin-ai-integration @karthikscale3
fix: add usage_metadata None checks to prevent NoneType errors (#429)
6cc2b23 
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: [email protected] <[email protected]>
@devin-ai-integration @karthikscale3
chore: bump version to 3.3.14 (#430)
8959561 
Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: [email protected] <[email protected]>
@obinnaokafor
add support for cohere client v2
45ae162
@obinnaokafor
add examples for v2 client for cohere
bf1ee17
@devin-ai-integration @karthikscale3
feat: add session ID support through HTTP headers
2197f22 
Co-Authored-By: [email protected] <[email protected]>
@devin-ai-integration @karthikscale3
test: add test cases for session ID functionality
0026efe 
Co-Authored-By: [email protected] <[email protected]>
@alizenhom
Merge pull request #426 from Scale3-Labs/obinna/S3EN-1111-dspy-bug-fix
ff5532d 
Ensure non-null check for  attribute
@obinnaokafor
bump version
82a9bc5
@alizenhom
Merge pull request #427 from Scale3-Labs/devin/1733790643-fix-base-ur…
1bc18e7 
…l-handling

fix: Make base_url handling consistent across all functions
@alizenhom
Merge pull request #432 from Scale3-Labs/obinna/S3EN-2920-update-cohe…
e0bc5e8 
…re-instrumentation

Obinna/s3 en 2920 update cohere instrumentation
@devin-ai-integration @karthikscale3
refactor: migrate session ID logic to get_headers function
346df1b 
Co-Authored-By: [email protected] <[email protected]>
@alizenhom
Merge pull request #433 from Scale3-Labs/devin/1733966293-add-session…
883d9d7 
…-id-support

feat: add session ID support through HTTP headers
@alizenhom
remove DSPY checkpoint tracing
ff0c46e
@alizenhom
Support exporter protocol
304c426
@alizenhom
version bump
820678a
@alizenhom
sync main
d854a06
@alizenhom
Merge pull request #438 from Scale3-Labs/ali/s3en-2927-customer-reque…
112d72a 
…stfix-signoz-issue

Ali/s3en 2927 customer requestfix signoz issue
@alizenhom alizenhom merged commit 3adae32 into main Dec 31, 2024
4 checks passed
@alizenhom alizenhom deleted the development branch December 31, 2024 13:54
@alizenhom alizenhom restored the development branch January 1, 2025 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alizenhom @obinnascale3 @obinnaokafor