-
Notifications
You must be signed in to change notification settings - Fork 1.9k
feat: add appinsights-instrumentation skill #525
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
base: main
Are you sure you want to change the base?
feat: add appinsights-instrumentation skill #525
Conversation
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.
Pull request overview
This pull request adds a new skill called "appinsights-instrumentation" that helps developers instrument web applications to send telemetry data to Azure Application Insights. The skill provides comprehensive guidance for multiple programming languages and hosting environments, along with supporting scripts and templates.
Key Changes
- Added a new skill structure under
skills/appinsights-instrumentation/with instructions for ASP.NET Core, Node.js, and Python applications - Included helper scripts (PowerShell Azure CLI commands) and Bicep templates for creating Application Insights resources
- Updated the skills README to register the new skill
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/appinsights-instrumentation/SKILL.md | Main skill documentation with guidelines for instrumenting webapps; includes prerequisites and context gathering instructions |
| skills/appinsights-instrumentation/ASPNETCORE.md | Instructions for instrumenting ASP.NET Core applications with Azure Monitor OpenTelemetry |
| skills/appinsights-instrumentation/AUTO.md | Guide for auto-instrumenting apps via Azure Portal without code changes |
| skills/appinsights-instrumentation/NODEJS.md | Instructions for instrumenting Node.js applications with Azure Monitor OpenTelemetry |
| skills/appinsights-instrumentation/PYTHON.md | Instructions for instrumenting Python applications with Azure Monitor OpenTelemetry |
| skills/appinsights-instrumentation/scripts/appinsights.ps1 | PowerShell script with Azure CLI commands for creating App Insights resources and configuring apps |
| skills/appinsights-instrumentation/examples/appinsights.bicep | Bicep template for provisioning Application Insights and Log Analytics Workspace |
| skills/appinsights-instrumentation/LICENSE.txt | MIT license file for the skill |
| docs/README.skills.md | Registry update to include the new appinsights-instrumentation skill |
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.Description
The skill includes instructions, Bicep templates and code samples to onboard a web app of supported kinds to send telemetry data to Azure App Insights. This skill aims at helping beginners to create add basic working telemetry points from which they can extend.
Type of Contribution
a new skill
Additional Notes
By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.