[AVM Question/Feedback]: Deprecation Notice enable_rbac_authorization

[mikhail-vasilchenko-bt]

Check for previous/existing GitHub issues

• I have checked for previous/existing GitHub issues

Description

Hi AVM team,

While creating a Key Vault using this module the output suggests to update property name. Should not be a big change and would be nice to get rid of that warning message.

│ Warning: Argument is deprecated
│
│ with azurerm_key_vault.this,
│ on main.tf line 7, in resource "azurerm_key_vault" "this":
│ 7: enable_rbac_authorization = !var.legacy_access_policies_enabled
│
│ This property has been renamed to rbac_authorization_enabled and will be
│ removed in v5.0 of the provider

Regards,
Mikhail

[matt-FFFFFF]

Hi! We will change this with the move to the new provider.

We don't support AzureRM 5 at the moment so this message is an annoyance but there is no danger of the it breaking.

[razorsedge]

This deprecation notice was introduced in the azurerm provider version 4.42.0.

[eclipse313]

This affects me too... Regardless of the provider version, whether v4 or v5, this should be easy to solve, as the old variable has been supported for some time now:

│ Warning: Argument is deprecated
│ 
│   with module.core.module.avm-res-keyvault-vault.azurerm_key_vault.this,
│   on .terraform/modules/core.avm-res-keyvault-vault/main.tf line 7, in resource "azurerm_key_vault" "this":
│    7:   enable_rbac_authorization       = !var.legacy_access_policies_enabled
│ 
│ This property has been renamed to `rbac_authorization_enabled` and will be removed in v5.0 of the provider

[joerianto83]

Hi i got the same issue

[matt-FFFFFF]

If we made this change we would need to raise the minimum version of AzureRM. This could be breaking for some so we would rather not do this for what amounts to a cosmetic change.

We will wait for AzureRM 5 and make the change then.