Skip to content

Commit

Permalink
feat: allow classType
Browse files Browse the repository at this point in the history
  • Loading branch information
shahargl committed Nov 11, 2024
1 parent af87a9c commit 4eb819b
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/keep/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v2
name: keep
version: 0.1.34
version: 0.1.35
description: Keep Helm Chart
type: application
icon: https://platform.keephq.dev/_next/image?url=%2Fkeep.png&w=48&q=75
Expand Down
2 changes: 1 addition & 1 deletion charts/keep/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ metadata:
labels:
{{- include "keep.labels" . | nindent 4 }}
annotations:
{{- if eq .Values.global.ingress.className "nginx" }}
{{- if or (eq .Values.global.ingress.className "nginx") (eq .Values.global.ingress.classType "nginx") }}
kubernetes.io/ingress.class: "nginx"
nginx.ingress.kubernetes.io/proxy-read-timeout: "3600"
nginx.ingress.kubernetes.io/proxy-send-timeout: "3600"
Expand Down
1 change: 1 addition & 0 deletions charts/keep/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ global:
ingress:
enabled: true
className: "nginx"
classType: "" # this is for cases where you want nginx configuration but the nginx className is different.

Check failure on line 15 in charts/keep/values.yaml

View workflow job for this annotation

GitHub Actions / lint-chart

15:19 [comments] too few spaces before comment
annotations: {}
# this is the prefix for the websocket route
# so the full url will be <ingress-host>/websocket
Expand Down

0 comments on commit 4eb819b

Please sign in to comment.