azurerm_key_vault_*: add TryCacheKeyvaultID function to speed up get keyvault id from keyvault URL

[wuxu92]

This is a short-term fixes #23404.

KeyVaultIDFromBaseUrl is time-consuming for listing almost all pages of the resources of the subscription to find the keyvault resource by name. This PR is trying to cache the mapping beween the keyvault URL and keyvault at first to void the ResourcesClient.List operation.

The process of finding the KeyVault resource by name using KeyVaultIDFromBaseUrl is time-consuming as it requires listing almost all pages of subscription resources. This PR aims to improve performance by caching the mapping between the KeyVault URL and its corresponding resource before the KeyVaultIDFromBaseUrl calls, thereby avoiding the need for a ResourcesClient.List operation. This PR won't speed up the terraform import.

[tombuildsstuff]

hey @wuxu92

Thanks for this PR

As mentioned in #23413 I've been looking into how the Resources API is used across the provider more broadly - and as a result have ended up opening #24019 which implements a fix for this in a different manner (by retrieving and caching all of the Key Vaults within a Subscription when populating the cache - rather than doing these one at a time).

Since I believe that #24019 will resolve #23404 in a slightly different manner, which'll alleviate the need for a short-term workaround for this one, whilst I'd like to thank you for this contribution I'm going to close this PR in favour of #24019 which should solve this in a different manner.

Thanks!

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.