dexidp / helm-charts

Dex Helm chart repository
Apache License 2.0
50 stars 82 forks source link

can't install chart when omiting pathType #145

Open gecube opened 3 days ago

gecube commented 3 days ago

Preflight Checklist

Chart

dex

Version

0.19.1

Expected Behavior

Install the chart with defaults

Actual Behavior

dex                         helmrelease/dex                         0.19.1      False       False   Helm install failed for release dex/dex with chart dex@0.19.1: 1 error occurred:                                                        
                                                                                                        * Ingress.networking.k8s.io "dex" is invalid: spec.rules[0].http.paths[0].pathType: Required value: pathType must be specified 
                                                                                                        *

Steps To Reproduce

prepare HelmRelease

apiVersion: helm.toolkit.fluxcd.io/v2beta1
kind: HelmRelease
metadata:
  name: dex
  namespace: dex
spec:
  install:
    remediation:
      retries: 3
  upgrade:
    remediation:
      retries: 3
  chart:
    spec:
      chart: dex
      sourceRef:
        kind: HelmRepository
        name: dex
      version: '*'
  interval: 1m0s
  values:
    ingress:
      enabled: true
      annotations:
        cert-manager.io/cluster-issuer: acme
      hosts:
        - host: ****
          paths:
            - path: /
      tls:
        - hosts:
            - ***
          secretName: dex-cert
      className: nginx
    ...

install

Additional Information

No response

Values

No response

gecube commented 3 days ago

or... one needs to update documentation here: https://github.com/dexidp/helm-charts/tree/master/charts/dex#minimal-configuration

and

here: https://github.com/dexidp/helm-charts/blob/fb4562eefb613fe9f94de1e367433aa8aec6e7ed/charts/dex/README.md?plain=1#L101