-
Notifications
You must be signed in to change notification settings - Fork 865
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
Support Application Insights querying #22355
Comments
Hello @JorTurFer can we get more info on what's broken in your scenario and how you're using |
Hi @jhendrixMSFT The main problem is that autorest uses it's own authorizer, which is not compatible with Thanks to the increase of covered services by azure-sdk-for-go, we have been able to migrate all the services from the old SDKs to the new ones based on this repo, which is nice because all of them are build with support for I have thought also in asking for support in Application Insights SDK for go, but it looks like abandoned (last commit was 2 years ago). The good point from |
Got it, thanks for clarifying. I don't know what our plans are for an AppInsights SDK. I've assigned this to @sandeep-sen for comments. |
Do you have any update @sandeep-sen ? 😄 |
+1 for this feature |
Service Package Request
Currently, we have SDK for multiple services, but Application Insights isn't included between them. In KEDA we are consolidating the Azure SDKs to use
github.com/Azure/azure-sdk-for-go
SDKs, as it has a huge improvement thanks to usejackfan.us.kg/Azure/azure-sdk-for-go/sdk/azidentity
.During the process, I've noticed that there isn't any way to directly query application insights and the only option that we have is to continue using
github.com/Azure/go-autorest/autorest
and managing the token providers by our side. After a review, I've found that for directly querying we should hitapi.applicationinsights.io
but this SDK only supports indirect queries viaapi.loganalytics.io
and it requires more information (like the resource group where the Insights is). Currently,github.com/Azure/go-autorest/autorest
just requires the InsightsID thanks to useapi.applicationinsights.io
The text was updated successfully, but these errors were encountered: