Azure / terraform-azurerm-avm-ptn-avd-lza-insights

AVM for Azure Virtual Desktop Insights
https://registry.terraform.io/modules/Azure/avm-ptn-avd-lza-insights/azurerm/latest
MIT License
3 stars 3 forks source link

[AVM Module Issue]: Support azurerm >4.0.0 #60

Closed audunsolemdal closed 2 months ago

audunsolemdal commented 2 months ago

Check for previous/existing GitHub issues

Issue Type?

Feature Request

(Optional) Module Version

0.1.2

(Optional) Correlation Id

No response

Description

Currently the azurerm provider is constrained to >= 3.71.0, < 4.0.0

My org is currently migrating to azurerm 4.x and this is putting a blocker.

I would propose that the constraint is changed to < 5.0.0

jensheerin commented 2 months ago

@audunsolemdal, thank you for opening issue. 4.0.0 testing is currently in progress and once completed the constraints will be adjusted.

jensheerin commented 2 months ago

62 added the subscription id requirement for version 4.0.0. Thank you again for the issue.

audunsolemdal commented 2 months ago

62 added the subscription id requirement for version 4.0.0. Thank you again for the issue.

Okay, but seems like the constraint in version 0.1.3 is still hindering me from using the module with azurerm 4.x https://github.com/Azure/terraform-azurerm-avm-ptn-avd-lza-insights/blob/main/terraform.tf#L3-L7

jensheerin commented 2 months ago

@audunsolemdal, thank you for following up. It is fixed now. There was a typo on the digit.