Skip to content

Reconfigure Application Insights #585

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
Closed
cecheta opened this issue Apr 4, 2024 · 0 comments · Fixed by #592
Closed

Reconfigure Application Insights #585

cecheta opened this issue Apr 4, 2024 · 0 comments · Fixed by #592
Assignees
@adamdougal
Labels
bug Something isn't working

Comments

@cecheta
Copy link
Collaborator

cecheta commented Apr 4, 2024 •
edited
Loading

Describe the bug

Application insights does not appear to be working correctly for the webapp.

  • When the app runs in App Service, app.py is not run directly, meaning that configure_azure_monitor() is never called.
  • According to this example, configure_azure_monitor() should be called before importing flask.

Expected behavior

We should be able to see the transaction map and traces in Application Insights.

Here is an example when configured correctly:

Screenshot 2024-04-04 111104

Debugging information

Steps to reproduce

  • Launch webapp
  • Make requests
  • View Application Insights

Screenshots

If applicable, add screenshots to help explain your problem.

Tasks

To be filled in by the engineer picking up the issue
@cecheta cecheta added the bug Something isn't working label Apr 4, 2024
@adamdougal adamdougal mentioned this issue Apr 4, 2024
Closed
4 tasks
@adamdougal adamdougal self-assigned this Apr 4, 2024
adamdougal added a commit that referenced this issue Apr 4, 2024
@adamdougal
Correctly configure app insights for the web app
3d1b82f 
Required by #585
@adamdougal adamdougal mentioned this issue Apr 4, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Apr 5, 2024
@adamdougal
Correctly configure app insights for the web app (#592)
470a1d1 
Required by #585
adamdougal added a commit that referenced this issue Apr 5, 2024
@adamdougal
Actually configure app insights correctly for web api
1ad04f6 
Although we were seeing the app, we were not seeing the requests. This
change fixes that issue.

Required by #585
@adamdougal adamdougal mentioned this issue Apr 5, 2024
Merged
github-merge-queue bot pushed a commit that referenced this issue Apr 5, 2024
@adamdougal
Actually configure app insights correctly for web api (#597)
256522f 
Although we were seeing the app, we were not seeing the requests. This
change fixes that issue.

Required by #585
eduardogch pushed a commit to devopsdale/chat-with-your-data-solution-accelerator that referenced this issue Apr 30, 2024
@adamdougal
Correctly configure app insights for the web app (Azure-Samples#592)
5e38cf5 
Required by Azure-Samples#585
eduardogch pushed a commit to devopsdale/chat-with-your-data-solution-accelerator that referenced this issue Apr 30, 2024
@adamdougal
Actually configure app insights correctly for web api (Azure-Samples#597
7e8ffe2 
)

Although we were seeing the app, we were not seeing the requests. This
change fixes that issue.

Required by Azure-Samples#585
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adamdougal adamdougal

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@adamdougal @cecheta