[AVM Module Issue]: Add an output to reference the provisioned KeyVault name

[nickmladenov]

Check for previous/existing GitHub issues

• I have checked for previous/existing GitHub issues

Issue Type?

Feature Request

(Optional) Module Version

No response

(Optional) Correlation Id

No response

Description

Could we introduce a resource_name output assigned the value azurerm_key_vault.this.name for this AVM?

This would avoid having to dissect the resource_id output through splitting by using a delimiter and magic numbers for improved code readability.

I've seen other AVM's declare resource as an output with the entire resource definition (azurerm_key_vault.this) specified to allow consumers to reference any missing attributes that have been custom defined.

I would raise a pull request feature for review including this change but it appears this AVM (like many others) has open contribution disabled as I am unable to create a new branch or push one up into the remote repository.

[matt-FFFFFF]

Hi,

We recently changed the specs to recommend against outputting the whole resource. It has issues with sensitive values and the potential for the schema to change

Instead we recommend that computed attributes are output.

Have you heard of basename()? It will give you the last part of the resource id..

Alternatively use split and reverse and take the zero index.

[nickmladenov]

Thanks for the recommendations @matt-FFFFFF, we have used similar in various places when data is available at those indexes although it isn't always the case.

I'll update our code with additional function usage in the meantime until there's an available output.

I completely agree with your reasoning for not outputting the whole resource as well as it reducing readability and IDE autocomplete. You mentioned the spec has been changes recently, has that already been published against the AVM standards and conventions document?