refactor: Restructure global ACME config and improve template logic

- Replaced `global.defaults` with `global.acme` for clarity and extensibility

Author: DrumRobot

README.md

@@ -33,7 +33,7 @@ helm install cluster-issuers ./ -f values.yaml
 
 ```yaml
 global:
-  defaults: # Optional
+  acme: # Defaults for all clusterIssuers below, can be overridden per issuer
     email: your-email@example.com
     solvers:
       - http01:
@@ -44,11 +44,12 @@ clusterIssuers:
   letsencrypt-prod:
     server: https://acme-v02.api.letsencrypt.org/directory
     privateKeySecret: letsencrypt-prod
+    # email, solvers will fallback to global.acme
 
   letsencrypt-staging:
-    email: staging@example.com
     server: https://acme-staging-v02.api.letsencrypt.org/directory
     privateKeySecret: letsencrypt-staging
+    email: staging@example.com # overrides global.acme.email
     solvers:
       - http01:
           ingress:

charts/cluster-issuers/templates/cluster-issuers.yaml

@@ -5,15 +5,15 @@ metadata:
   name: {{ $name }}
 spec:
   acme:
-    email: {{ default $.Values.global.email $issuer.email }}
+    email: {{ default $.Values.global.acme.email $issuer.email }}
     server: {{ $issuer.server }}
     privateKeySecretRef:
       name: {{ $issuer.privateKeySecret }}
     solvers:
 {{- if hasKey $issuer "solvers" }}
-{{ toYaml $issuer.solvers | indent 6 }}
-{{- else if hasKey $.Values.global "solvers" }}
-{{ toYaml $.Values.global.solvers | indent 6 }}
+{{ toYaml $issuer.solvers | indent 4 }}
+{{- else if hasKey $.Values.global.acme "solvers" }}
+{{ toYaml $.Values.global.acme.solvers | indent 4 }}
 {{- else }}
     - http01:
         ingress: