Add Azure Local skills#2163
Open
tfrankland2 wants to merge 14 commits into
Open
Conversation
Author
@microsoft-github-policy-service agree company="Microsoft" |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new azure-local skill to the Azure skills plugin, providing Azure Local (Azure Stack HCI) deployment/operations/workload/security/update/troubleshooting guidance, and wires it into the scheduled integration test skill list.
Changes:
- Added a new
plugin/skills/azure-local/skill with core routing plus workflow and reference markdown. - Registered
azure-localintests/skills.jsonand included it in the integration schedule. - Updated
plugin/README.mdto mention Azure Local under Compute & Containers.
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/skills.json | Adds azure-local to the skill registry and scheduled integration run list. |
| plugin/skills/azure-local/SKILL.md | Introduces the Azure Local skill frontmatter + top-level routing/rules/refs. |
| plugin/skills/azure-local/version.json | Adds per-skill NBGV version configuration. |
| plugin/skills/azure-local/workflows/plan-and-deploy/plan-and-deploy.md | Deployment planning/intake and guarded deployment sequencing. |
| plugin/skills/azure-local/workflows/operate-and-update/operate-and-update.md | Ops + lifecycle update flow, including update safety guidance and inventory patterns. |
| plugin/skills/azure-local/workflows/workload-management/workload-management.md | VM/AKS hybrid/workload management routing and safety checks. |
| plugin/skills/azure-local/workflows/networking-and-security/networking-and-security.md | Networking + security flows, SDN routing, and guardrails. |
| plugin/skills/azure-local/workflows/troubleshooting/troubleshooting.md | Triage flow, scenario routing, evidence checklist, and safety rules for remediation. |
| plugin/skills/azure-local/references/docs-map.md | Curated Microsoft Learn URL map for version-aware Azure Local documentation. |
| plugin/skills/azure-local/references/mcp-and-cli-tools.md | Guidance on MCP/ARG-first discovery plus safe CLI/PowerShell usage boundaries. |
| plugin/skills/azure-local/references/resource-types.md | Azure Resource Graph resource type patterns and baseline KQL inventory queries. |
| plugin/skills/azure-local/references/safety-rules.md | Centralized “ask before doing” and high-risk component handling rules. |
| plugin/README.md | Mentions Azure Local in the “Available Tools” coverage list. |
cesquedamsft
reviewed
May 5, 2026
jongio
reviewed
May 6, 2026
Collaborator
jongio
left a comment
jongio
left a comment
There was a problem hiding this comment.
Clean new skill addition for Azure Local with good workflow separation and safety guardrails. A few suggestions to bring the SKILL.md into full compliance with the repo's skill-files.instructions.md.
jongio
reviewed
May 6, 2026
Collaborator
jongio
left a comment
jongio
left a comment
There was a problem hiding this comment.
Addresses my previous feedback. The SKILL.md sections now follow the repo's table format conventions and docs-map.md uses latest-version URLs by default. @cesquedamsft ready for your final approval.
Member
|
Looks good to me. Approved. |
jongio
reviewed
May 7, 2026
Collaborator
|
@tfrankland2 this has merge conflicts now - can you rebase on main? All my prior feedback is addressed and CI is green, so just the conflict left to sort out. |
Add a dedicated Azure Local skill with workflows and references for deployment, operations, workloads, networking, security, updates, and troubleshooting. Update adjacent Azure skills to route Azure Local, Arc VM, AKS hybrid, and lifecycle scenarios to the new skill. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add azure-local to the skill test schedule, update affected trigger snapshots, and remove cross-skill relative markdown links that escaped skill directories. Shorten the azure-upgrade description so its metadata unit test remains within the existing word limit. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove extra plugin manifest keywords and avoid treating custom locations as an Azure Local-owned routing signal in adjacent skills. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Remove the azure-upgrade routing change and clarify that Azure MCP support for Azure Local is partial through generic Azure surfaces rather than a dedicated Azure Local tool namespace. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Keep the Azure Local PR focused on adding the new skill and required registration, without modifying existing skill routing. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Align Azure Local Resource Graph examples with existing resource lookup guidance, which describes ARG as a KQL subset. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Address PR review feedback by adding Azure Local unit, trigger, and integration tests, standardizing the top-level skill sections, and keeping the skill entrypoint under the token limit. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use an inline quoted description so the skill passes the frontmatter validator used by CI. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
tfrankland2
force-pushed
the
feature/azure-local-skill
branch
from
67be1c4 to
8bbac0d
Compare
tfrankland2
requested review from
MadhuraBharadwaj-MSFT,
alex-thompson,
rakal-dyh and
saikoumudi
as code owners
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
tfrankland2
force-pushed
the
feature/azure-local-skill
branch
from
8bbac0d to
d922943
Compare
Author
|
Oops sorry if you got tagged for review. Forced pushed some changes by accident while rebasing. Should be fixed now @jongio @JasonYeMSFT |
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
jongio
approved these changes
May 13, 2026
Collaborator
jongio
left a comment
jongio
left a comment
There was a problem hiding this comment.
All prior feedback addressed - tables for Quick Reference/MCP Tools/Error Handling, version pinning removed from docs-map, README wording reverted. Skill structure looks solid with good progressive disclosure across references and workflows. Safety rules are thorough.
Only remaining blocker is the merge conflict from the rebase - once that's resolved this is good to go.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes #2164
Validation