azurerm_key_vault
data source: cache the vault_uri
once read
#17407
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Mitigate #11059.
THe listing resources API won't be called if the
keyVaultCache
is populated. Currently, the cache is populated in following places (prior to calling the resource listing API):The reports in #11059 are mostly happening when the depending key vault is a data source. This PR adds support for that use case.
In result, as long as users guarantee the correct dependency between key vault (either resource or data source) and the nested items, then the listing resource API is avoided.