-
Notifications
You must be signed in to change notification settings - Fork 924
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
[trace][bugfix] Check both OTel environ for exporter to PFS #3274
Conversation
promptflow SDK CLI Azure E2E Test Result zhengfei/bugfix/add-otel 4 files 4 suites 4m 12s ⏱️ Results for commit ac8cd1e. |
SDK CLI Global Config Test Result zhengfei/bugfix/add-otel6 tests 6 ✅ 1m 15s ⏱️ Results for commit ac8cd1e. |
Executor Unit Test Result zhengfei/bugfix/add-otel796 tests 796 ✅ 3m 45s ⏱️ Results for commit ac8cd1e. |
Executor E2E Test Result zhengfei/bugfix/add-otel246 tests 240 ✅ 5m 1s ⏱️ Results for commit ac8cd1e. |
SDK CLI Test Result zhengfei/bugfix/add-otel 4 files 4 suites 1h 4m 30s ⏱️ Results for commit ac8cd1e. |
Description
New SDK will set different environ (
OTEL_EXPORTER_OTLP_TRACES_ENDPOINT
), as consumer, need to check both to avoid breaking change.All Promptflow Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines