feat: make ClusterIssuer optional to allow certmanager defaults

armadaproject / armada-operator

Apache License 2.0

13 stars 10 forks source link

feat: make ClusterIssuer optional to allow certmanager defaults #299

Closed jpoler closed 6 months ago

jpoler commented 7 months ago

Certmanager is capable of setting it's own default issuer. Armada operator requires that .Spec.ClusterIssuer is supplied and hardcodes cluster issuer annotations. This precludes the ability to rely on certmanager defaults.

This PR makes ClusterIssuer optional, and only attaches cluster issuer annotations when ClusterIssuer is supplied.

dejanzele commented 7 months ago

Thanks for the PR @jpoler, I'll test it out now