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

Provision script should enable app insights for luis and web app #3093

Closed
benbrown opened this issue May 15, 2020 · 5 comments · Fixed by #3425
Closed

Provision script should enable app insights for luis and web app #3093

benbrown opened this issue May 15, 2020 · 5 comments · Fixed by #3425
Assignees
Labels
R10 Release 10 - August 17th, 2020

Comments

@benbrown
Copy link
Contributor

When a bot is configured via the Azure portal, it configures app insights on the web app and LUIS instance.

Our current provision process does not configure these settings.

If possible, we should make it so that a bot provisioned via Composer is configured in the same way it would be if done via the portal.

@benbrown benbrown added Type: Enhancement Needs-triage A new issue that require triage labels May 15, 2020
@goergenj
Copy link

Adding detail on what's missing:

  • On AppInsights an API-Key need to be created to read the telemetry from the Bot Service.
  • On Bot Service add AppInsights settings on the "settings" page.
  • On Bot WebApp activate AppInsights and configure to use the same AppInsights instance as the Bot Services.

Yes, it should be the same experience as when creating a "Web App Bot" via the Azure Portal.

@zidaneymar
Copy link
Contributor

When a bot is configured via the Azure portal, it configures app insights on the web app and LUIS instance.

Our current provision process does not configure these settings.

If possible, we should make it so that a bot provisioned via Composer is configured in the same way it would be if done via the portal.

Hi @benbrown , currently the provision would create a app insights service on Azure and generate a instrumentation key in appsettings.deployment.json and this key would be used to initialize a telemetry client in runtime
User could open this app insights service on portal to see the details.
image

image

@xieofxie
Copy link
Contributor

@benbrown
Copy link
Contributor Author

Yes, we should set the bot service analytics as well as the webapp service.

@cwhitten cwhitten added R10 Release 10 - August 17th, 2020 and removed Needs-triage A new issue that require triage labels May 24, 2020
@zidaneymar
Copy link
Contributor

I didn't find a way to link luis with application insights but link the bot service and app insights based on @xieofxie 's comment in PR #3425

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R10 Release 10 - August 17th, 2020
Projects
None yet
5 participants