Fix bug where KeyVault with ARMID owner can't be recovered

Azure / azure-service-operator

Azure Service Operator allows you to create Azure resources using kubectl

https://azure.github.io/azure-service-operator/

MIT License

702 stars 188 forks source link

Fix bug where KeyVault with ARMID owner can't be recovered #4127

Closed matthchr closed 1 week ago

matthchr commented 1 week ago

Fixes #4117.

Also fix an issue where we gave a confusing and non-fatal error when attempting to recover a KeyVault in a different ResourceGroup than it was originally in. Changed the error to be clearer and also immediately fatal so that no retries happen.

If applicable:

[ ] this PR contains documentation
[x] this PR contains tests
[ ] this PR contains YAML Samples

matthchr commented 1 week ago

/ok-to-test sha=8e92842

matthchr commented 1 week ago

/ok-to-test sha=27f11b4