Remove the extra whitespace around the supplied (or ommitted) exclude regexes

[wallrj]

Before:

$ helm template deploy/charts/venafi-kubernetes-agent   --show-only templates/configmap.yaml | yq '.data."config.yaml"' | head
cluster_id: ""
cluster_description: ""
server: "https://api.venafi.cloud/"
period: "0h1m0s"
exclude-annotation-keys-regex:

  []
exclude-label-keys-regex:

  []

$ helm template deploy/charts/venafi-kubernetes-agent   --show-only templates/configmap.yaml --set config.excludeAnnotationKeysRegex={'.*'}   --set config.excludeLabelKeysRegex={'.*'} | yq '.data."conf
ig.yaml"' | head
cluster_id: ""
cluster_description: ""
server: "https://api.venafi.cloud/"
period: "0h1m0s"
exclude-annotation-keys-regex:

  - .*
exclude-label-keys-regex:

  - .*

After:

$ helm template deploy/charts/venafi-kubernetes-agent   --show-only templates/configmap.yaml  | yq '.data."config.yaml"' | head
cluster_id: ""
cluster_description: ""
server: "https://api.venafi.cloud/"
period: "0h1m0s"
venafi-cloud:
  uploader_id: "no"
  upload_path: "/v1/tlspk/upload/clusterdata"
data-gatherers:
# gather k8s apiserver version information
- kind: "k8s-discovery"

$ helm template deploy/charts/venafi-kubernetes-agent   --show-only templates/configmap.yaml  --set config.excludeAnnotationKeysRegex={'.*'}   --set config.excludeLabelKeysRegex={'.*'} | yq '.data."config.yaml"' | head
cluster_id: ""
cluster_description: ""
server: "https://api.venafi.cloud/"
period: "0h1m0s"
exclude-annotation-keys-regex:
  - .*
exclude-label-keys-regex:
  - .*
venafi-cloud:
  uploader_id: "no"

Testing

I ran the e2e test and it succeeded:

$ make test-e2e-gke
...
{
  "ts": 1731686855541.627,
  "caller": "agent/run.go:224",
  "msg": "Failed to complete initial sync of DataGatherer",
  "logger": "Run",
  "kind": "k8s-dynamic",
  "name": "k8s/fireflyissuers",
  "err": "timed out waiting for Kubernetes caches to sync"
}
{"ts":1731686856218.3735,"caller":"agent/run.go:409","msg":"Data sent successfully","v":0,"logger":"Run.gatherAndOutputData.postData"}

Then I got the resulting configmap and saw that it contained the new supplied annotation exclusion, but that the excludeLabel field was omitted, as expected, which did not cause any problems:

$ kubectl get cm -n venafi agent-config  -o yaml  | yq '.data."config.yaml"'  | head
cluster_id: "venafi-kubernetes-agent-e2e"
cluster_description: "A kind cluster used for testing the venafi-kubernetes-agent.\n"
server: "https://api.venafi.eu/"
period: "0h1m0s"
exclude-annotation-keys-regex:
  - ^kapp\.k14s\.io/original.*
venafi-cloud:
  uploader_id: "no"
  upload_path: "/v1/tlspk/upload/clusterdata"
data-gatherers:

[maelvls]

Thanks for the thorough testing, as well as the well-written description! I haven't tested the changes myself. I am confident that nothing will break, though, thanks to your extensive testing.