helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

can't install chart when omiting pathType

Open gecube opened this issue 1 year ago • 2 comments

Preflight Checklist

  • [X] I agree to follow the Code of Conduct that this project adheres to.
  • [X] I have searched the issue tracker for an issue that matches the one I want to file, without success.
  • [X] I am not looking for support or already pursued the available support channels without success.

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 [email protected]: 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 avatar Oct 02 '24 18:10 gecube

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

gecube avatar Oct 02 '24 18:10 gecube

Seems like the defaults aren't working as expected https://github.com/dexidp/helm-charts/blob/65db60c3c27b2124db53792eeab80a574588b92c/charts/dex/values.yaml#L210-L214

clementblaise avatar Jun 16 '25 15:06 clementblaise