Kuadrant / dns-operator

Kuadrant DNS Operator
Apache License 2.0
1 stars 7 forks source link

Remove ManagedZone API #166

Closed maleck13 closed 1 month ago

maleck13 commented 3 months ago

What

Originally ManagedZone was thought of in the context of multiple tenants where we might control the credentals and the zones. This is no longer the case. ManagedZone as remained in place largely as a reference to a credential to the underlying DNS provider. We do not need this indirection. Additionally the ManagedZone API adds some confusion around what should happen when you delete a managed zone resource.

How

Remove the ManagedZone resource and update DNSPolicy to use a provider ref to define a specific secret in that namespace. The dns record controller will propagate this secret to the DNS Record.

These changes to DNS Operator will break kuadrant-operator, so have both PRs ready and approved, before merging the DNS Operator changes.

See also https://github.com/Kuadrant/multicluster-gateway-controller/issues/406

maleck13 commented 3 months ago

@mikenairn would appreciate any addiitional thoughts you have on this one.

maleck13 commented 1 month ago

https://github.com/Kuadrant/kuadrant-operator/pull/793