Skip to content

docs: add MAAS provisioning skill - #1377

Merged
michael-balint merged 4 commits into
masterfrom
dholt/add-provision-with-maas-skill
Aug 13, 2026
Merged

docs: add MAAS provisioning skill#1377
michael-balint merged 4 commits into
masterfrom
dholt/add-provision-with-maas-skill

Conversation

@dholt

@dholt dholt commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Add the provision-with-maas operator skill

What this is

Create the missing public DeepOps skills/provision-with-maas/SKILL.md using only current repository behavior in docs/pxe/maas.md, config.example/maas-inventory.yml, scripts/maas_inventory.py, scripts/maas_deploy.sh, and playbooks/provisioning/maas.yml. Cover preconditions, MAAS tags to dynamic inventory, deploy, post-provision validation, reruns, cleanup, and observed failure branches. Do not access MAAS, a cluster, or any network; do not claim live validation. Update skills/README.md. Keep this documentation-only and local; do not push or open a PR.

Verification

  • Deterministic checks passed: diff check, skill structure, python compile, shell syntax, public sanitizer, parent commit
  • Two independent agent reviews approved head c583aa035c09 (different model vendors; strict verdict contract)
  • Published by the DeepOps maintenance loop's deterministic publisher after its publication gate (reviews, provenance, exact-head) passed

@dholt
dholt requested a review from michael-balint August 12, 2026 22:58
michael-balint
michael-balint previously approved these changes Aug 13, 2026
@dholt

dholt commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

Verified the branch update: it is a clean integration of current master (which now includes the deploy-airgapped skill from #1376). The MAAS skill content is byte-identical to the previously reviewed head c583aa0; the only delta in skills/README.md is the deploy-airgapped table row inherited from master. All 30 checks are green on the updated head. Good to merge from my side.

(Verification performed by the maintenance automation on Doug's behalf.)

@dholt

dholt commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

@michael-balint the merge is blocked only because this repo has dismiss_stale_reviews enabled: the "Update branch" merge you pushed automatically dismissed your own earlier approval (dismissed 16:02 UTC, the moment the branch updated). There is no code-owner or last-pusher restriction in play — submitting a fresh Approve review unblocks the merge button immediately. Content is verified unchanged from the reviewed head (see the comment above) and checks are 30/30 green.

(Diagnosis by the maintenance automation on Doug's behalf — branch-protection settings checked via the API.)

Integrates the deploy-airgapped skill row (#1376) alongside the MAAS skill
row in skills/README.md; identical resolution to the prior branch update,
re-pushed by the PR author so the reviewer's approval survives.
@dholt
dholt force-pushed the dholt/add-provision-with-maas-skill branch from 2a5fdb8 to 857ad77 Compare August 13, 2026 19:34
@dholt

dholt commented Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

@michael-balint replaced the branch-update merge with an author-pushed commit carrying a byte-identical tree to your conflict resolution (git diff between the two heads is empty). With the last push now coming from the PR author, a single Approve from you should stick and the merge button will unblock once checks finish re-running (~15 minutes). Sorry for the settings runaround — dismiss_stale_reviews plus a reviewer-side branch update turns out to be a trap.

(Executed by the maintenance automation on Doug's behalf; content is unchanged from the reviewed head.)

@dholt
dholt requested a review from michael-balint August 13, 2026 19:50
@michael-balint
michael-balint merged commit 6c2760b into master Aug 13, 2026
30 checks passed
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.

2 participants