-
Notifications
You must be signed in to change notification settings - Fork 94
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
Create workspace based app insights #2619
Conversation
What's the error? If you increase the sku, that will usually resolve the issue.
Yeah, there's nothing to really pick and as far as I know, no benefit to really having a bunch of workspaces. I'll confirm with the App Service team, but that is the intended behavior. You must have had one created with a function app at some point. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checked Advanced and compared resources against the docs, so LGTM. I'll double check Basic if it becomes available, but approving so as not to block you.
Are you using the free tier? |
It reproduces on all the tiers offered via quick picks |
Leverages log analytics create step, which is required to migrate App Insights to the Workspace Based. Here is some documentation on that. This was already being done in Functions, so the step already existed.
It also uses the connection string property rather than the instrumentational key since that is the recommended path going forward.
Another reference: https://learn.microsoft.com/en-us/azure/azure-monitor/app/create-workspace-resource
How to test: