Skip to content

feat: an Azure infrastructure consultant - #2

Merged
craigthackerx merged 1 commit into
mainfrom
feat/azure-infra-consultant
Aug 24, 2026
Merged

feat: an Azure infrastructure consultant#2
craigthackerx merged 1 commit into
mainfrom
feat/azure-infra-consultant

Conversation

@craigthackerx

Copy link
Copy Markdown
Contributor

Copilot already knows Azure. What it does not know is your position, so that is what this agent carries.

The house position it enforces

  • Check for a published house module before proposing raw resources. You publish over a hundred Terraform modules. The agent names one when it exists, and says so plainly when it cannot confirm one does, 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. 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. Learned the hard way in this repo's sibling earlier today.
  • Names built inside the module from structured inputs so a caller cannot drift them; tags constructed once and merged, never per resource.

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.yaml so 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 validate clean: 8 agents, 0 warnings.

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.
@craigthackerx
craigthackerx merged commit e7d2cdc into main Aug 24, 2026
3 checks passed
@craigthackerx
craigthackerx deleted the feat/azure-infra-consultant branch August 24, 2026 22:06
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.

1 participant