fix(namespace): set location as top-level attribute on azapi_resource

[Copilot]

Creating a managed namespace via the module failed with 400 LocationRequired because the namespace sub-module only embedded location inside the request body of azapi_resource, which the provider does not forward as the resource's location.

Changes

• modules/namespace/main.tf: set location = var.location as a top-level attribute on azapi_resource.this, matching the pattern used by the parent cluster in main.tf.
• modules/namespace/locals.tf: drop the now-redundant location key from local.resource_body.

resource "azapi_resource" "this" {
  location  = var.location   # <-- added
  name      = var.name
  parent_id = var.parent_id
  type      = "Microsoft.ContainerService/managedClusters/managedNamespaces@2025-10-01"
  body      = local.resource_body
  ...
}

The user-reported config (namespace = { ns1 = { name = "...", location = "germanywestcentral", annotations = {...} } }) already plumbs location through main.namespace.tf into the sub-module, so no caller-side changes are needed.

[microsoft-github-policy-service]

Important

The "Needs: Triage 🔍" label must be removed once the triage process is complete!

Tip

For additional guidance on how to triage this issue/PR, see the Terraform Issue Triage documentation.

[matt-FFFFFF]

@copilot run PORCH_NO_TUI=1 ./avm pre-commit and commit changes so the tests pass

[Copilot]

@copilot run PORCH_NO_TUI=1 ./avm pre-commit and commit changes so the tests pass

Ran PORCH_NO_TUI=1 ./avm pre-commit, all steps passed (avmfix, generate docs, grept). Committed the resulting governance-sync changes (avm script update, AVM skill folder rename to avm-terraform-module-development, AGENTS.md reference update) in a938690.