What this PR does / why we need it:
The error handling on updating DNS providers is fixed for the following edge case: There is an error on reading the secret for the external provider. In this case the error is only recorded and the externalDNSProvider is deleted.
The correct behaviour is to stop processing and return the error.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:
Fail if reading secret for external provider fails.
How to categorize this PR?
/area control-plane /kind bug
What this PR does / why we need it: The error handling on updating DNS providers is fixed for the following edge case: There is an error on reading the secret for the
external
provider. In this case the error is only recorded and theexternal
DNSProvider
is deleted. The correct behaviour is to stop processing and return the error.Which issue(s) this PR fixes: Fixes #
Special notes for your reviewer:
Release note: