seriohub / velero-helm

Helm charts for velero-ui, velero-api, and velero-watchdog
6 stars 3 forks source link

Remove redundant tls hosts / secret config #26

Closed dcaputo-harmoni closed 1 month ago

dcaputo-harmoni commented 1 month ago

This change is to remove the redundant config for tls certs. Previously, the ingress pulls two tls hosts / secrets:

Method 1:

uiIngress:
  tls:
    enabled: true

Method 2:

uiIngress:
  spec:
    tls:
      - hosts:
        - yourhost.com
        secretName: your-secret-name

But only one is needed, and if you specify both it will create redundant certificates. Removing the simplified method and keeping the more configurable one.