Skip to content

Introduce AKS review skill#2076

Open
johnbilliris wants to merge 5 commits intomicrosoft:mainfrom
johnbilliris:azure-kubernetes-review
Open

Introduce AKS review skill#2076
johnbilliris wants to merge 5 commits intomicrosoft:mainfrom
johnbilliris:azure-kubernetes-review

Conversation

@johnbilliris
Copy link
Copy Markdown

Description

Introduce an Azure Kubernetes Review skill

Checklist

  • Tests pass locally (cd tests && npm test)
  • If modifying skill descriptions: verified routing correctness with integration tests (npm run test:skills:integration -- <skill>)
  • If modifying skill USE FOR / DO NOT USE FOR / PREFER OVER clauses: confirmed no routing regressions for competing skills
  • Version bumped in skill frontmatter (if skill files changed)

Related Issues

Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings April 28, 2026 06:26
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

Introduces a new azure-kubernetes-review skill under plugin/skills/ to guide an evidence-based AKS posture review and generate a standardized Markdown audit report.

Changes:

  • Adds the azure-kubernetes-review skill with an end-to-end workflow and guardrails.
  • Adds reference documents (checklist matrix, container checks, diagnostics detector mapping) plus a report template.
  • Updates .gitignore to ignore *.crt files.

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 11 comments.

Show a summary per file
File Description
plugin/skills/azure-kubernetes-review/SKILL.md Skill entrypoint, workflow outline, inputs/outputs, guardrails
plugin/skills/azure-kubernetes-review/assets/aks-audit-report-template.md Report template for audit output
plugin/skills/azure-kubernetes-review/references/aks-checklist-matrix.md Canonical checklist matrix + validation commands
plugin/skills/azure-kubernetes-review/references/aks-container-best-practices.md Container-level best-practice checks and scoring rules
plugin/skills/azure-kubernetes-review/references/aks-diagnostics-detectors.md Diagnostics detector extraction + mapping guidance
plugin/skills/azure-kubernetes-review/references/quality-gates.md Completeness checks and decision logic
plugin/skills/azure-kubernetes-review/references/workflow-scope-and-checklist.md Scope definition + checklist loading workflow
plugin/skills/azure-kubernetes-review/references/workflow-diagnostics.md Diagnostics execution workflow
plugin/skills/azure-kubernetes-review/references/workflow-warning-events.md Warning event collection + correlation workflow
plugin/skills/azure-kubernetes-review/references/workflow-container-assessment.md Container assessment workflow
plugin/skills/azure-kubernetes-review/references/workflow-validation-and-findings.md Validation command execution + findings workflow
plugin/skills/azure-kubernetes-review/references/workflow-report.md Report generation workflow
.gitignore Ignores *.crt files

Comment thread plugin/skills/azure-kubernetes-review/SKILL.md Outdated
Comment thread plugin/skills/azure-kubernetes-review/SKILL.md
Comment thread plugin/skills/azure-kubernetes-review/references/aks-checklist-matrix.md Outdated
Comment thread plugin/skills/azure-kubernetes-review/SKILL.md Outdated
Comment thread plugin/skills/azure-kubernetes-review/references/aks-checklist-matrix.md Outdated
Comment thread plugin/skills/azure-kubernetes-review/assets/aks-audit-report-template.md Outdated
johnbilliris and others added 3 commits April 28, 2026 16:39
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Collaborator

@kvenkatrajan kvenkatrajan left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution! Consider consolidating this with the main azure-kubernetes skill. We are trying to keep the number of skills low and reduce overlap between skills

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.

3 participants