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

Open Telemetry - fixes #243 #1285 #1262

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

karpikpl
Copy link

@karpikpl karpikpl commented Feb 8, 2025

Motivation and Context

Fixes #243 #1285

Description

Added Open Telemetry integration with Application Insights

Contribution Checklist

  • I have built and tested the code locally and in a deployed app
  • For frontend changes, I have pulled the latest code from main, built the frontend, and committed all static files.
  • This is a change for all users of this app. No code or asset is specific to my use case or my organization.
  • I didn't break any existing functionality 😄

Working app with azure search
Screenshot 2025-02-25 195215

@karpikpl
Copy link
Author

not sure why tests are failing. But I'm also not getting what I want from open telemetry.
It would be nice if we could instrument the frontend.

@karpikpl
Copy link
Author

@anushree1808 could you help with this PR? I'm not really sure why Python tests are failing in the build, it looks like they execute against an LLM but I don't think this would be affected by logging?

@karpikpl karpikpl changed the title Open Telemetry - fixes #243 Open Telemetry - fixes #243 #1285 Feb 25, 2025
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.

Application Insights not usable when deployed as webapp from AI Studio
1 participant