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

2.0 - App Insights integration #3389

Closed
tdykstra opened this issue May 19, 2017 · 17 comments
Closed

2.0 - App Insights integration #3389

tdykstra opened this issue May 19, 2017 · 17 comments
Assignees
Labels
Pri1 High priority, do before Pri2 and Pri3
Milestone

Comments

@tdykstra
Copy link
Contributor

tdykstra commented May 19, 2017

App Insights integration

Hosting startup and AI

@tdykstra tdykstra added this to the 2017-2.0 RTW milestone May 19, 2017
@tdykstra tdykstra changed the title Azure integration in 2.0 2.0 - DI from environment, App Insights integration Jul 10, 2017
@Rick-Anderson Rick-Anderson added the Pri1 High priority, do before Pri2 and Pri3 label Jul 21, 2017
@tdykstra tdykstra self-assigned this Aug 3, 2017
@tdykstra tdykstra changed the title 2.0 - DI from environment, App Insights integration 2.0 - App Insights integration Aug 11, 2017
@tdykstra
Copy link
Contributor Author

Assigned to @bwren

@ghost
Copy link

ghost commented Aug 12, 2017

Where can I read details about this?

I have found only this info:
Hosting environments can now inject extra package dependencies and execute code during application
startup, without the application needing to explicitly take a dependency or call any methods. This feature can be used to enable certain environments to "light-up" features unique to that environment without the application needing to know ahead of time.

In ASP.NET Core 2.0, this feature is used to automatically enable Application Insights diagnostics when debugging in Visual Studio and (after opting in) when running in Azure App Services. As a result, the project templates no longer add Application Insights packages and code by default.

@tdykstra
Copy link
Contributor Author

@HardHub [@]bwren will be documenting the Application Insights integration. No ETA yet.

@scottaddie scottaddie assigned scottaddie and unassigned tdykstra Sep 12, 2017
@bitbonk
Copy link

bitbonk commented Sep 16, 2017

Until the docs are done, where can I learn how this works? Any hint?

@normj
Copy link

normj commented Sep 18, 2017

I found this blog post interesting in understanding how the injection works. http://www.hishambinateya.com/istartup-vs-ihostingstartup

@Rick-Anderson
Copy link
Contributor

@bwren can we help you get started on this? Do you have an ETA for this.

@Rick-Anderson Rick-Anderson mentioned this issue Sep 25, 2017
14 tasks
@bwren
Copy link

bwren commented Sep 25, 2017

@Rick-Anderson Sorry, there was some confusion on this. Adding @mrbullwinkle who now owns App Insights. I'll work with him to get this addressed.

@scottaddie
Copy link
Member

@mrbullwinkle Are there any updates to provide on this? Anything you need from me?

@mrbullwinkle
Copy link
Contributor

@scottaddie it is on my internal VSTS board and I had originally planned to have it done this month. It got bumped by a few other tasks that I had not anticipated. I have two days blocked out next week to start working on this. I will send you an update 11/3 which I am tentatively targeting to have a first draft for your feedback/review.

@scottaddie
Copy link
Member

@mrbullwinkle Just following up on this and hoping to get a status. Anything I can do to help?

@Rick-Anderson
Copy link
Contributor

Rick-Anderson commented Dec 15, 2017

@Rick-Anderson
Copy link
Contributor

cc @cecilphillip

@Rick-Anderson
Copy link
Contributor

@mrbullwinkle can you give us an ETA on this? @scottaddie is willing to help.

@mrbullwinkle
Copy link
Contributor

@Rick-Anderson we have new App Insights content coming out for .Net Core next week: 1/22-1/26 and 1/29-31. Goal is to have all the content requests from your team complete by month end.

@scottaddie
Copy link
Member

@mrbullwinkle Any updates to provide?

@mrbullwinkle
Copy link
Contributor

@scottaddie Apologies again for the previous delay. I am working on the new App Insights .NET Core content right now,

@mrbullwinkle
Copy link
Contributor

This issue is addressed in this doc: https://docs.microsoft.com/azure/application-insights/app-insights-asp-net-core

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pri1 High priority, do before Pri2 and Pri3
Projects
None yet
Development

No branches or pull requests

7 participants