Skip to content

Gap-6: App Service Diagnose (B → A) — Dedicated troubleshooting guide#1640

Merged
kvenkatrajan merged 17 commits intomainfrom
pass-equity-gap-6
Apr 29, 2026
Merged

Gap-6: App Service Diagnose (B → A) — Dedicated troubleshooting guide#1640
kvenkatrajan merged 17 commits intomainfrom
pass-equity-gap-6

Conversation

@paulyuk
Copy link
Copy Markdown
Member

@paulyuk paulyuk commented Apr 1, 2026

Closes #1614 | Parent: #1608

1 file: app-service/README.md — High CPU, deployment failures, crashes, slow responses, TLS issues. MCP tool usage guide. KQL queries. Symptom/cause/fix tables.

Starting assessment — domain experts should review.

Closes #1614 | Parent: #1608
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 1, 2026 17:37
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a dedicated App Service troubleshooting reference covering common App Service symptoms, diagnostic commands (Azure CLI/Kudu), KQL queries, and MCP tool guidance to support “Gap-6: App Service Diagnose (B → A)”.

Changes:

  • Introduces a symptom → cause → quick fix matrix for common App Service failures.
  • Adds step-by-step sections with Azure CLI commands and KQL queries for CPU/memory, deployments, crashes, latency, and TLS/custom domains.
  • Documents relevant MCP tools and includes a “combined diagnostic script” snippet.

Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2026

Details# 🔍 Token Analysis Report

@github-copilot-for-azure/scripts@1.0.0 tokens
node --import tsx src/tokens/cli.ts compare --base origin/main --head HEAD --markdown

fatal: path 'plugin/skills/azure-diagnostics/references/app-service/README.md' exists on disk, but not in 'origin/main'

📊 Token Change Report

Comparing origin/mainHEAD

Summary

Metric Value
📈 Total Change +1,970 tokens (+152%)
Before 1,293 tokens
After 3,263 tokens
Files Changed 2

Changed Files

File Before After Change
plugin/skills/azure-diagnostics/references/app-service/README.md - 1,840 +1840
plugin/skills/azure-diagnostics/SKILL.md 1,293 1,423 +130 (+10%)

@github-copilot-for-azure/scripts@1.0.0 tokens
node --import tsx src/tokens/cli.ts check --markdown

📊 Token Limit Check Report

Checked: 575 files
Exceeded: 87 files

⚠️ Files Exceeding Token Limits

File Tokens Limit Over By
.github/skills/analyze-skill-issues/SKILL.md 2109 500 +1609
.github/skills/analyze-test-run/SKILL.md 2471 500 +1971
.github/skills/file-test-bug/SKILL.md 628 500 +128
.github/skills/sensei/README.md 3531 2000 +1531
.github/skills/sensei/SKILL.md 3026 500 +2526
.github/skills/sensei/references/EXAMPLES.md 3701 2000 +1701
.github/skills/sensei/references/LOOP.md 4181 2000 +2181
.github/skills/sensei/references/SCORING.md 4299 2000 +2299
.github/skills/skill-authoring/SKILL.md 839 500 +339
plugin/skills/airunway-aks-setup/SKILL.md 1025 500 +525
plugin/skills/appinsights-instrumentation/SKILL.md 911 500 +411
plugin/skills/azure-ai/SKILL.md 820 500 +320
plugin/skills/azure-aigateway/SKILL.md 1261 500 +761
plugin/skills/azure-aigateway/references/policies.md 2342 2000 +342
plugin/skills/azure-cloud-migrate/SKILL.md 848 500 +348
plugin/skills/azure-cloud-migrate/references/services/container-apps/cloudrun-deployment-guide.md 2029 2000 +29
plugin/skills/azure-cloud-migrate/references/services/container-apps/deployment-guide.md 2458 2000 +458
plugin/skills/azure-cloud-migrate/references/services/container-apps/fargate-deployment-guide.md 2587 2000 +587
plugin/skills/azure-cloud-migrate/references/services/functions/lambda-to-functions.md 2600 2000 +600
plugin/skills/azure-cloud-migrate/references/services/functions/runtimes/javascript.md 2181 2000 +181
plugin/skills/azure-compliance/SKILL.md 1188 500 +688
plugin/skills/azure-compute/SKILL.md 1090 500 +590
plugin/skills/azure-compute/workflows/vm-recommender/vm-recommender.md 2631 2000 +631
plugin/skills/azure-compute/workflows/vm-troubleshooter/vm-troubleshooter.md 2509 2000 +509
plugin/skills/azure-cost/SKILL.md 1980 500 +1480
plugin/skills/azure-deploy/SKILL.md 1645 500 +1145
plugin/skills/azure-deploy/references/pre-deploy-checklist.md 4095 2000 +2095
plugin/skills/azure-deploy/references/recipes/azd/errors.md 4004 2000 +2004
plugin/skills/azure-deploy/references/troubleshooting.md 2038 2000 +38
plugin/skills/azure-diagnostics/SKILL.md 1423 500 +923
plugin/skills/azure-enterprise-infra-planner/SKILL.md 1002 500 +502
plugin/skills/azure-enterprise-infra-planner/references/constraints/compute-apps.md 2022 2000 +22
plugin/skills/azure-hosted-copilot-sdk/SKILL.md 1263 500 +763
plugin/skills/azure-kubernetes/SKILL.md 2606 500 +2106
plugin/skills/azure-kubernetes/azure-kubernetes-automatic-readiness/SKILL.md 3609 500 +3109
plugin/skills/azure-kusto/SKILL.md 2152 500 +1652
plugin/skills/azure-messaging/SKILL.md 821 500 +321
plugin/skills/azure-prepare/SKILL.md 3359 500 +2859
plugin/skills/azure-prepare/references/aspire.md 4617 2000 +2617
plugin/skills/azure-prepare/references/plan-template.md 2617 2000 +617
plugin/skills/azure-prepare/references/recipes/azd/aspire.md 2275 2000 +275
plugin/skills/azure-prepare/references/recipes/azd/terraform.md 3555 2000 +1555
plugin/skills/azure-prepare/references/research.md 2274 2000 +274
plugin/skills/azure-prepare/references/resources-limits-quotas.md 3322 2000 +1322
plugin/skills/azure-prepare/references/security.md 2147 2000 +147
plugin/skills/azure-prepare/references/services/functions/bicep.md 3127 2000 +1127
plugin/skills/azure-prepare/references/services/functions/templates/recipes/composition.md 2813 2000 +813
plugin/skills/azure-prepare/references/services/functions/terraform.md 3404 2000 +1404
plugin/skills/azure-prepare/references/services/sql-database/bicep.md 2037 2000 +37
plugin/skills/azure-quotas/SKILL.md 2821 500 +2321
plugin/skills/azure-quotas/references/commands.md 2644 2000 +644
plugin/skills/azure-resource-lookup/SKILL.md 1394 500 +894
plugin/skills/azure-resource-visualizer/SKILL.md 2122 500 +1622
plugin/skills/azure-storage/SKILL.md 1228 500 +728
plugin/skills/azure-upgrade/SKILL.md 1249 500 +749
plugin/skills/azure-upgrade/references/languages/java/INSTRUCTION.md 2724 2000 +724
plugin/skills/azure-upgrade/references/languages/java/package-specific/com.microsoft.azure.management.md 2215 2000 +215
plugin/skills/azure-upgrade/references/languages/java/templates/PLAN_TEMPLATE.md 2411 2000 +411
plugin/skills/azure-upgrade/references/languages/java/templates/PROGRESS_TEMPLATE.md 2315 2000 +315
plugin/skills/azure-upgrade/references/languages/java/templates/SUMMARY_TEMPLATE.md 2190 2000 +190
plugin/skills/azure-upgrade/references/services/functions/automation.md 3463 2000 +1463
plugin/skills/azure-upgrade/references/services/functions/consumption-to-flex.md 2773 2000 +773
plugin/skills/azure-validate/SKILL.md 950 500 +450
plugin/skills/entra-agent-id/SKILL.md 4001 500 +3501
plugin/skills/entra-app-registration/SKILL.md 2070 500 +1570
plugin/skills/entra-app-registration/references/api-permissions.md 2545 2000 +545
plugin/skills/entra-app-registration/references/cli-commands.md 2211 2000 +211
plugin/skills/entra-app-registration/references/console-app-example.md 2752 2000 +752
plugin/skills/entra-app-registration/references/oauth-flows.md 2375 2000 +375
plugin/skills/microsoft-foundry/SKILL.md 2939 500 +2439
plugin/skills/microsoft-foundry/foundry-agent/create/create.md 4315 2000 +2315
plugin/skills/microsoft-foundry/foundry-agent/create/references/toolbox.md 2802 2000 +802
plugin/skills/microsoft-foundry/foundry-agent/deploy/deploy.md 5894 2000 +3894
plugin/skills/microsoft-foundry/foundry-agent/eval-datasets/eval-datasets.md 2344 2000 +344
plugin/skills/microsoft-foundry/foundry-agent/eval-datasets/references/trace-to-dataset.md 4268 2000 +2268
plugin/skills/microsoft-foundry/foundry-agent/invoke/invoke.md 2084 2000 +84
plugin/skills/microsoft-foundry/foundry-agent/observe/observe.md 2542 2000 +542
plugin/skills/microsoft-foundry/foundry-agent/trace/references/kql-templates.md 2701 2000 +701
plugin/skills/microsoft-foundry/models/deploy-model/SKILL.md 1640 500 +1140
plugin/skills/microsoft-foundry/models/deploy-model/capacity/SKILL.md 1739 500 +1239
plugin/skills/microsoft-foundry/models/deploy-model/customize/SKILL.md 2235 500 +1735
plugin/skills/microsoft-foundry/models/deploy-model/customize/references/customize-workflow.md 3335 2000 +1335
plugin/skills/microsoft-foundry/models/deploy-model/preset/SKILL.md 1226 500 +726
plugin/skills/microsoft-foundry/models/deploy-model/preset/references/preset-workflow.md 5534 2000 +3534
plugin/skills/microsoft-foundry/quota/quota.md 2288 2000 +288
plugin/skills/microsoft-foundry/quota/references/capacity-planning.md 2080 2000 +80
plugin/skills/microsoft-foundry/references/sdk/foundry-sdk-py.md 2162 2000 +162

Consider moving content to references/ subdirectories.


Automated token analysis. See skill authoring guidelines for best practices.

Copy link
Copy Markdown
Collaborator

@jongio jongio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two CI checks are failing because SKILL.md was not updated alongside this new reference file:\n\n1. Skill Structure - metadata.version needs a bump (1.0.4 -> next). CI requires this when any file under a skill changes.\n\n2. Markdown References - references/app-service/README.md is not linked from SKILL.md. Add an App Service row to the "Troubleshooting Guides by Service" table and a link in the References section at the bottom.\n\nBeyond the CI fixes - the SKILL.md description and "Triggers" section do not mention "App Service" at all. Without adding trigger phrases like "troubleshoot app service", "app service high CPU", "app service deployment failure", this skill will not activate when users ask about App Service issues - making the new guide undiscoverable.\n\nThe guide content itself is solid - good symptom/cause/fix matrices, practical KQL queries, and the combined diagnostic script is a nice touch. Worth also addressing the copilot-bot comments on table formatting, curl quoting, and MCP tool naming.

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 22, 2026

Two CI checks are failing because SKILL.md was not updated alongside this new reference file:\n\n1. Skill Structure - metadata.version needs a bump (1.0.4 -> next). CI requires this when any fil...

Addressed in commit 38d1842. I bumped metadata.version to 1.0.5, added App Service to the Troubleshooting Guides table, linked references/app-service/README.md in References, and added App Service trigger phrases in both frontmatter description and Triggers. I also aligned MCP tool naming and updated the processes example to use ARM + Entra ID auth.

Copilot AI review requested due to automatic review settings April 22, 2026 23:39
@paulyuk paulyuk review requested due to automatic review settings April 22, 2026 23:39
Copilot AI review requested due to automatic review settings April 23, 2026 19:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Comment thread plugin/skills/azure-diagnostics/SKILL.md Outdated
Comment thread plugin/skills/azure-diagnostics/SKILL.md
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 23, 2026 20:03
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/SKILL.md
Comment thread plugin/skills/azure-diagnostics/SKILL.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 23, 2026 20:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/SKILL.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Address MCP Tools table, not all scenarios have an MCP tool and the calls are expensive.  Changed table to offer MCP or AZ CLI commands as applicable

Co-authored-by: Andrew Westgarth <mail@hawaythelads.co.uk>
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Comment thread plugin/skills/azure-diagnostics/references/app-service/README.md Outdated
Correcting command for diagnosis and querying of logs

Co-authored-by: Andrew Westgarth <mail@hawaythelads.co.uk>
JasonYeMSFT
JasonYeMSFT previously approved these changes Apr 27, 2026
@apwestgarth apwestgarth requested a review from jongio April 28, 2026 16:44
jongio
jongio previously approved these changes Apr 28, 2026
Copy link
Copy Markdown
Collaborator

@jongio jongio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My earlier feedback (version placeholder, SKILL.md linking, trigger phrases) - all addressed. CI is green.

The trigger test update properly distinguishes troubleshooting from provisioning intent. The troubleshooting guide has been through detailed domain review with @apwestgarth and @JasonYeMSFT.

Content looks good - practical symptom/cause/action matrices, correct CLI examples, and useful KQL queries.

Updated snapshot to include additional troubleshooting keywords for Azure diagnostics to clean merge.
@apwestgarth apwestgarth dismissed stale reviews from jongio and JasonYeMSFT via b7fcae2 April 29, 2026 18:04
Expanded the description to include additional troubleshooting scenarios for Azure services.
Copy link
Copy Markdown
Collaborator

@jongio jongio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addresses my previous feedback. SKILL.md now includes App Service trigger phrases and references, snapshots are updated, and test naming is clearer. The guide content has been refined through thorough domain expert review. Looks good.

@kvenkatrajan kvenkatrajan merged commit 9a3cf56 into main Apr 29, 2026
12 checks passed
@kvenkatrajan kvenkatrajan deleted the pass-equity-gap-6 branch April 29, 2026 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Gap-6: App Service Diagnose (B → A) — Dedicated troubleshooting guide + MCP tool usage

7 participants