Skip to content

fix: missing location parameter in namespace request#162

Merged
matt-FFFFFF merged 6 commits intomainfrom
fork-pr-159-fix/missing-location-parameter-in-namespace-request
Feb 24, 2026
Merged

fix: missing location parameter in namespace request#162
matt-FFFFFF merged 6 commits intomainfrom
fork-pr-159-fix/missing-location-parameter-in-namespace-request

Conversation

@matt-FFFFFF
Copy link
Copy Markdown
Member

This PR replaces #159 to enable CI/CD checks with required secrets.

Original PR by @jonas-budde


Description

Hi, i got this error when using the submodule "namespace":

│ Error: Failed to create/update resource
│ 
│   with module.namespace.azapi_resource.this,
│   on .terraform/modules/namespace/main.tf line 1, in resource "azapi_resource" "this":
│    1: resource "azapi_resource" "this" {
│ 
│ creating/updating Resource: (ResourceId
│ "/subscriptions/xyz/resourceGroups/rg-aks-gwc-dev/providers/Microsoft.ContainerService/managedClusters/aks-xyz-gwc-dev/managedNamespaces/xyz"
│ / Api Version "2025-10-01"): PUT
│ https://management.azure.com/subscriptions/xyz/resourceGroups/rg-aks-gwc-dev/providers/Microsoft.ContainerService/managedClusters/aks-xyz-gwc-dev/managedNamespaces/xyz
│ --------------------------------------------------------------------------------
│ RESPONSE 400: 400 Bad Request
│ ERROR CODE: LocationRequired
│ --------------------------------------------------------------------------------
│ {
│   "error": {
│     "code": "LocationRequired",
│     "message": "The location property is required for this definition."
│   }
│ }
│ --------------------------------------------------------------------------------
│ 

The API apprently has changed and now needs the location as a property in the request body.

Type of Change

  • Non-module change (e.g. CI/CD, documentation, etc.)
  • Azure Verified Module updates:
    • Bugfix containing backwards compatible bug fixes
      • Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
      • The bug was found by the module author, and no one has opened an issue to report it yet.
    • Feature update backwards compatible feature updates.
    • Breaking changes.
    • Update to documentation

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • I did run all pre-commit checks

- Add examples/namespaces/ with AKS Automatic cluster deploying two
  managed namespaces demonstrating annotations, labels, network
  policies, resource quotas, adoption policy, and delete policy.
- Make default_resource_quota required in the namespace submodule
  since the Azure API rejects requests without it (returns 400:
  'resource quota cannot be nil') despite the API spec marking it
  as optional.
- Regenerate docs via avm pre-commit.
@matt-FFFFFF matt-FFFFFF merged commit 49b84ab into main Feb 24, 2026
8 of 16 checks passed
@matt-FFFFFF matt-FFFFFF deleted the fork-pr-159-fix/missing-location-parameter-in-namespace-request branch February 24, 2026 17:02
@matt-FFFFFF matt-FFFFFF temporarily deployed to empty-no-approval March 26, 2026 16:57 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Triage 🔍 Maintainers need to triage still

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants