search

galaxyproject / galaxy-helm

Minimal setup required to run Galaxy under Kubernetes
MIT License
41 stars 39 forks source link

Suggestion about galaxy/ingress.yaml #368

Closed mshaikh786 closed 2 years ago

mshaikh786 commented 2 years ago

At times one may need to dedicate an ingressclass and corresponding ingress-controller. In the galaxy/templates/ingress.yaml, the ingressClassName is set to "nginx" which is usually the default class name when an ingress-controller is deployed. Would it be better to set the same variable as is set in the galaxy/templates/ingress-activity-canary.yaml, i.e. ingressClassName: {{ .Values.ingress.ingressClassName }}

nuwang commented 2 years ago

Thanks for reporting this. This is a bug and should be fixed.