Skip to content

docs(storage-fsx-windows-sla-optimizer): link the deployable demo - #116

Merged
ams-thakkar merged 1 commit into
aws:mainfrom
benlec:docs/fsx-sla-skill-demo-link
Oct 1, 2026
Merged

ams-thakkar merged 1 commit into
aws:mainfrom
benlec:docs/fsx-sla-skill-demo-link

Conversation

@benlec

@benlec benlec commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a "Try it in a deployable demo" section to the storage-fsx-windows-sla-optimizer README, pointing at the demo that showcases this skill in the GenAI Ops Demo Library: resilience/fsx-windows-sla-review-devops-agent.

The demo deploys an Agent Space with this skill registered (through the Asset API), a Single-AZ FSx for Windows file system joined to a self-managed Active Directory, its alarms, and a Lab that injects three reversible mis-configurations (rotated AD credentials, backups disabled, alarm removed) and records what the agent concludes with and without the skill. The demo references the skill at a stated ref of this repository and never copies it; its README links back here. This PR closes the loop in the other direction.

README-only change: no skill content, evals or front matter touched. The eval evidence already present in the skill directory keeps the PR check green.

Testing

  • Demo deployed and the three scenarios run end to end in eu-central-1 (the credentials scenario produced an investigation that loaded this skill and named invalid service-account credentials as the cause; the two Chat scenarios produced the review sentences quoted in the demo's lab/scenarios.yaml).
  • Rendered the Markdown locally; links verified.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ams-thakkar ams-thakkar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approving. Every claim in the 19 added lines is falsifiable, and I checked them against the demo rather than reading for plausibility: the link returns 200, all three scenario headings match lab/scenarios.yaml exactly, each carries an autoRevertSeconds and a Rollback path so "reversible" holds, and MISCONFIGURED, FreeStorageCapacity, Single-AZ, the self-managed AD and "about an hour" all match the demo's own README.

The one that mattered most to me — "references the skill at a stated ref of this repository and never copies it" — is true: shared/devops-agent/agent-tools/deploy-skill.sh sparse-clones this repo and zips the skill directory, and deploy-all.sh passes --ref main explicitly. No vendored copy anywhere in that repo, so the demo can't drift from what we ship here.

"With and without this skill" is recorded as a table column per scenario in the demo README, so that line is accurate too.

Repo side: mkdocs build --strict clean, the new link is absolute and consistent with the earlier absolute-link fixes in bc50051 and 6075d1e, no CHANGELOG bump needed for a README-only change per ample precedent, and zero commits behind main.

Thanks for pointing the skill at something people can actually run.

@ams-thakkar
ams-thakkar merged commit fc5b777 into aws:main Oct 1, 2026
1 check 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