Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ingressClassName missing #29

Open
vikiscience opened this issue Jan 26, 2025 · 0 comments
Open

ingressClassName missing #29

vikiscience opened this issue Jan 26, 2025 · 0 comments

Comments

@vikiscience
Copy link

vikiscience commented Jan 26, 2025

Hi All, I'm trying to install the langflow-ide chart (version 0.1.1) on a Kubernetes cluster (v1.30), and have a problem to get ingress working.

The only way to specify a non-default ingress class in the chart is to use the ingress annotation kubernetes.io/ingress.class. However, this annotation is deprecated since Kubernetes 1.18, and the field ingressClassName should be used instead. Unfortunately, this field is missing from the manifest, and thus the ingress does not work.

Can this field please be added to the chart?

Suggested change in the ingress manifest:

spec:
  {{- with .Values.ingress.className }}
  ingressClassName: {{ . }}
  {{- end }}

Or else, for legacy clusters, the ingress manifest could look analogous to this:
https://github.com/langflow-ai/langflow-helm-charts/blob/main/charts/langflow-runtime/templates/ingress.yaml#L26-L28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant