feat: an Azure infrastructure consultant - #2
Merged
Conversation
Copilot already knows Azure. What it does not know is the house position, so that is what this agent carries rather than general service knowledge. Check for a published house module before proposing raw resources, and say plainly when a module cannot be confirmed rather than inventing an address that will not resolve. Managed identity, then OIDC, then nothing: a stored client secret is a finding, not a design. Private by default, and honest about what that breaks, because a deployment-package storage account still has to be writable by whatever runs the deployment and a trusted-services bypass does not cover a build agent. Names built inside the module from structured inputs; tags constructed once and merged. Two rules do the heavy lifting. The five Well-Architected pillars are a review lens and the agent must NAME THE TRADE, because a recommendation claiming all five are satisfied is usually one that has not been thought about. And it must NEVER QUOTE A PRICE: pricing is regional, changes without notice and depends on commitments it cannot see, so it states the meter, what drives it and which choice moves it, then points at the calculator. A confident wrong number is worse than none, because someone will budget against it. It also separates three failures people collapse into one word: zone redundancy, region pairing and backup answer an instance dying, a region failing, and someone deleting the data. Four knowledge packs added, the CAF ones from the public cloud-adoption-framework mirror. The Well-Architected Framework is deliberately NOT a pack: it has no reliable public markdown mirror and the one that exists is a 2019 archive, so the pillars live in the instructions and the detail comes from scoped search against the live pages. Shipping a stale archive as authoritative would have been worse than not shipping it. Budget 7513/8000.
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.
Copilot already knows Azure. What it does not know is your position, so that is what this agent carries.
The house position it enforces
The two rules doing the heavy lifting
Name the trade. Every design is assessed against the five Well-Architected pillars and the agent must say which it traded away. A recommendation claiming all five are satisfied is usually one that has not been thought about.
Never quote a price. Pricing is regional, changes without notice, and depends on commitments the agent cannot see. It states the cost model (the meter, what drives it, which choice moves it) then points at the calculator. A confident wrong number is worse than no number, because someone will budget against it.
It also separates three failures people collapse into one word: zone redundancy, region pairing and backup answer an instance dying, a region failing, and someone deleting the data.
A sourcing decision worth flagging
The Well-Architected Framework is deliberately not a knowledge pack. There is no reliable public markdown mirror of it. The one that turns up (
MicrosoftDocs/architecture-sandbox) is dated 20 November 2019.So the five pillars live in the instructions as a review lens, and the detail comes from scoped web search against the live pages. Shipping a six-year-old archive as authoritative knowledge would have been worse than not shipping it, and the reasoning is recorded in
sources.yamlso nobody adds it later without knowing.CAF is public and current, so the landing zone design areas, resource naming and abbreviations are packs.
Gates
Budget 7513/8000, under the warning line.
just validateclean: 8 agents, 0 warnings.