Skip to content

Update module.tf#62

Open
florianow wants to merge 1 commit intomainfrom
feature/pin-azapi-version
Open

Update module.tf#62
florianow wants to merge 1 commit intomainfrom
feature/pin-azapi-version

Conversation

@florianow
Copy link
Copy Markdown
Contributor

someone have to test the module with a newer version of azapi. It does not work anymore on newer version.

Root Cause: The azapi provider version 2.9.0 has changed behavior compared to earlier versions. In newer versions of azapi, the output attribute of azapi_resource_list returns a native Terraform object instead of a JSON string. The Problem:

  • Lines 49-52 in the module use jsondecode() on data.azapi_resource_list.billing_role_definitions.output
  • But in azapi v2.x, .output is already an object, not a JSON string
  • The error message confirms this: data.azapi_resource_list.billing_role_definitions.output is object with 1 attribute "value" This is a breaking change introduced in azapi provider 2.0. The module was written for azapi 1.x where .output was a JSON string that needed decoding.

someone have to test the module with a newer version of azapi. It does not work anymore on newer version.

Root Cause: The azapi provider version 2.9.0 has changed behavior compared to earlier versions. In newer versions of azapi, the output attribute of azapi_resource_list returns a native Terraform object instead of a JSON string.
The Problem:
- Lines 49-52 in the module use jsondecode() on data.azapi_resource_list.billing_role_definitions.output
- But in azapi v2.x, .output is already an object, not a JSON string
- The error message confirms this: data.azapi_resource_list.billing_role_definitions.output is object with 1 attribute "value"
This is a breaking change introduced in azapi provider 2.0. The module was written for azapi 1.x where .output was a JSON string that needed decoding.
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