-
Notifications
You must be signed in to change notification settings - Fork 88
Create top-level AKS folder and SKILL.md #1029
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?
Changes from 25 commits
edd2004
a4eab8e
2cf0363
da83ce2
59186b0
ac9301a
f24eb8e
16c29c8
9dc9578
278d7a0
6e2ab85
3f6e3a6
3428b30
35c636c
4995afd
2f940d5
1a92efd
d58b49b
fc92679
f63b19d
b47bed8
a2acfc2
1b8e483
2b11b8c
4a0a598
7dc8f22
0862041
77758cd
a74df39
9ac4bda
e7e5c15
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,124 @@ | ||||||
| --- | ||||||
| name: azure-kubernetes | ||||||
| description: "Plan and create production-ready Azure Kubernetes Service (AKS) clusters. Covers Day-0 decisions and Day-1 configuration, cluster SKUs (Automatic vs Standard), security, monitoring, reliability/performance best practices, upgrades, and networking. WHEN: create AKS cluster, plan AKS configuration, design AKS networking, AKS Automatic vs Standard, AKS security, AKS upgrade strategy, AKS autoscaling, AKS monitoring setup, AKS cost analysis, Day-0 checklist." | ||||||
| --- | ||||||
julia-yin marked this conversation as resolved.
Show resolved
Hide resolved
julia-yin marked this conversation as resolved.
Show resolved
Hide resolved
Comment on lines
+1
to
+7
|
||||||
|
|
||||||
| # Azure Kubernetes Service | ||||||
|
|
||||||
| > **AUTHORITATIVE GUIDANCE — MANDATORY COMPLIANCE** | ||||||
| > | ||||||
| > This skill produces a **recommended AKS cluster configuration** based on user requirements, distinguishing **Day-0 decisions** (networking, API server — hard to change later) from **Day-1 features** (can enable post-creation). See [CLI reference](./references/cli-reference.md) for commands. | ||||||
|
|
||||||
julia-yin marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
| ## Quick Reference | ||||||
| | Property | Value | | ||||||
| |----------|-------| | ||||||
| | Best for | AKS cluster planning and Day-0 decisions | | ||||||
| | MCP Tools | `mcp_azure_mcp_aks`, `mcp_aks_mcp_az_aks_operations` | | ||||||
|
||||||
| | MCP Tools | `mcp_azure_mcp_aks`, `mcp_aks_mcp_az_aks_operations` | | |
| | MCP Tools | `mcp_azure_mcp_aks`, `mcp_azure_mcp_az_aks_operations` | |
Copilot
AI
Mar 3, 2026
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.
The MCP tool names in Quick Reference (mcp_azure_mcp_aks, mcp_aks_mcp_az_aks_operations) don’t match the MCP tool naming used elsewhere in this repo (e.g., aks is referenced as the dedicated AKS MCP tool in azure-resource-lookup). These identifiers also aren’t referenced anywhere else in the repo, so they’re likely incorrect. Please align this row with the actual tool name(s) used by the AKS MCP server (or remove the row if tool support isn’t available).
Copilot
AI
Mar 2, 2026
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.
This SKILL.md doesn’t follow the repository’s Skill File Authoring Guidelines required section structure (Quick Reference, When to Use This Skill, MCP Tools, Workflow/Steps, Error Handling). Please restructure the document to include those sections/tables so it’s consistent with other plugin skills and easier to scan.
Copilot
AI
Feb 25, 2026
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.
Rule 2 refers to an "AKS MCP server", but repo MCP config only defines a generic azure MCP server (plugin/.mcp.json). This will lead agents to look for a non-existent server; please update the rule to reference the Azure MCP server and the relevant AKS-related MCP tools (or CLI) explicitly.
| 2. Use the AKS MCP server for invoking Azure API and kubectl commands when applicable during the cluster setup and operations processes. | |
| 2. Use the `azure` MCP server and its AKS-related MCP tools to invoke Azure APIs and perform AKS and kubectl operations whenever possible during cluster setup and ongoing operations; if required functionality is not available via MCP tools, fall back to Azure CLI and kubectl commands. |
julia-yin marked this conversation as resolved.
Show resolved
Hide resolved
Copilot
AI
Mar 3, 2026
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.
Missing required MCP Tools section: the repo’s skill authoring guidelines require an explicit “MCP Tools” section with a table of commands/parameters (not just listing tool names in Quick Reference). See .github/instructions/skill-files.instructions.md (Required Sections #3).
julia-yin marked this conversation as resolved.
Show resolved
Hide resolved
julia-yin marked this conversation as resolved.
Show resolved
Hide resolved
julia-yin marked this conversation as resolved.
Show resolved
Hide resolved
julia-yin marked this conversation as resolved.
Show resolved
Hide resolved
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| # CLI Reference for AKS | ||
|
|
||
| ```bash | ||
| # List AKS clusters | ||
| az aks list --output table | ||
|
|
||
| # Show cluster details | ||
| az aks show --name CLUSTER --resource-group RG | ||
|
|
||
| # Get available Kubernetes versions | ||
| az aks get-versions --location LOCATION --output table | ||
|
|
||
| # Create AKS Automatic cluster | ||
| az aks create --name CLUSTER --resource-group RG --sku automatic \ | ||
| --network-plugin azure --network-plugin-mode overlay \ | ||
| --enable-oidc-issuer --enable-workload-identity | ||
|
|
||
| # Create AKS Standard cluster | ||
| az aks create --name CLUSTER --resource-group RG \ | ||
| --node-count 3 --zones 1 2 3 \ | ||
| --network-plugin azure --network-plugin-mode overlay \ | ||
| --enable-cluster-autoscaler --min-count 1 --max-count 10 | ||
|
|
||
| # Get credentials | ||
| az aks get-credentials --name CLUSTER --resource-group RG | ||
|
|
||
| # List node pools | ||
| az aks nodepool list --cluster-name CLUSTER --resource-group RG --output table | ||
|
|
||
| # Enable monitoring | ||
| az aks enable-addons --name CLUSTER --resource-group RG \ | ||
| --addons monitoring --workspace-resource-id WORKSPACE_ID | ||
julia-yin marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| ``` | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,103 @@ | ||
| // Jest Snapshot v1, https://goo.gl/fbAQLP | ||
|
|
||
| exports[`azure-kubernetes - Trigger Tests Trigger Keywords Snapshot skill description triggers match snapshot 1`] = ` | ||
| { | ||
| "description": "Plan and create production-ready Azure Kubernetes Service (AKS) clusters. Covers Day-0 decisions and Day-1 configuration, cluster SKUs (Automatic vs Standard), security, monitoring, reliability/performance best practices, upgrades, and networking. WHEN: create AKS cluster, plan AKS configuration, design AKS networking, AKS Automatic vs Standard, AKS security, AKS upgrade strategy, AKS autoscaling, AKS monitoring setup, AKS cost analysis, Day-0 checklist.", | ||
| "extractedKeywords": [ | ||
| "aks", | ||
| "analysis", | ||
| "automatic", | ||
| "autoscaling", | ||
| "azure", | ||
| "best", | ||
| "checklist", | ||
| "cli", | ||
| "cluster", | ||
| "clusters", | ||
| "configuration", | ||
| "container", | ||
| "cost", | ||
| "covers", | ||
| "create", | ||
| "day-0", | ||
| "day-1", | ||
| "decisions", | ||
| "deploy", | ||
| "design", | ||
| "diagnostic", | ||
| "entra", | ||
| "identity", | ||
| "key vault", | ||
| "kubernetes", | ||
| "mcp", | ||
| "monitor", | ||
| "monitoring", | ||
| "networking", | ||
| "observability", | ||
| "performance", | ||
| "plan", | ||
| "practices", | ||
| "production-ready", | ||
| "reliability", | ||
| "security", | ||
| "service", | ||
| "setup", | ||
| "skus", | ||
| "standard", | ||
| "strategy", | ||
| "upgrade", | ||
| "upgrades", | ||
| "when", | ||
| ], | ||
| "name": "azure-kubernetes", | ||
| } | ||
| `; | ||
|
|
||
| exports[`azure-kubernetes - Trigger Tests Trigger Keywords Snapshot skill keywords match snapshot 1`] = ` | ||
| [ | ||
| "aks", | ||
| "analysis", | ||
| "automatic", | ||
| "autoscaling", | ||
| "azure", | ||
| "best", | ||
| "checklist", | ||
| "cli", | ||
| "cluster", | ||
| "clusters", | ||
| "configuration", | ||
| "container", | ||
| "cost", | ||
| "covers", | ||
| "create", | ||
| "day-0", | ||
| "day-1", | ||
| "decisions", | ||
| "deploy", | ||
| "design", | ||
| "diagnostic", | ||
| "entra", | ||
| "identity", | ||
| "key vault", | ||
| "kubernetes", | ||
| "mcp", | ||
| "monitor", | ||
| "monitoring", | ||
| "networking", | ||
| "observability", | ||
| "performance", | ||
| "plan", | ||
| "practices", | ||
| "production-ready", | ||
| "reliability", | ||
| "security", | ||
| "service", | ||
| "setup", | ||
| "skus", | ||
| "standard", | ||
| "strategy", | ||
| "upgrade", | ||
| "upgrades", | ||
| "when", | ||
| ] | ||
| `; |
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.
SKILLSis a bash array here;SKILLS="${SKILLS# }"converts it into a scalar string (first element only), sonpm run frontmatter -- "${SKILLS[@]}"will no longer validate all changed skills. Remove this line, or (if you intended to trim) operate on a separate scalar variable and keepSKILLSas an array.