Skip to content

Commit fe69420

Browse files
authored
expose lightning healthcheck endpoint (#109)
* expose lightning healthcheck endpoint * fixing hassim's findings * adding judy env vars * update chart version
1 parent d236cab commit fe69420

File tree

4 files changed

+46
-7
lines changed

4 files changed

+46
-7
lines changed

charts/multiwoven/Chart.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ description: |
44
Multiwoven is an open-source reverse ETL tool, offering an alternative to qHightouch, Census, and similar platforms. 🔥
55
# kubeVersion: ">=1.16.0"
66
type: application
7-
version: 0.72.0
8-
appVersion: "0.72.0"
7+
version: 0.73.0
8+
appVersion: "0.73.0"
99
home: https://github.com/Multiwoven/multiwoven
1010
sources:
1111
- https://docs.squared.ai/open-source/guides/setup/helm

charts/multiwoven/templates/lightning-deployment.yaml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,15 @@ spec:
6666
value: {{ .Values.lightningConfig.seemoreVlm | quote }}
6767
- name: SEEMORE_ENDPOINT
6868
value: {{ .Values.lightningConfig.seemoreEndpoint | quote }}
69+
70+
- name: JUDY_API_KEY
71+
value: {{ .Values.lightningConfig.judyApiKey | quote }}
72+
- name: JUDY_MODEL
73+
value: {{ .Values.lightningConfig.judyModel | quote }}
74+
- name: JUDY_ENDPOINT
75+
value: {{ .Values.lightningConfig.judyEndpoint | quote }}
6976
image: {{ .Values.lightning.lightning.image.repository }}:{{ .Values.lightning.lightning.image.tag | default .Chart.AppVersion }}
77+
# command: ["sleep", "3600"] # Uncomment for debugging
7078
securityContext:
7179
runAsUser: {{ .Values.lightning.lightning.containerSecurityContext.runAsUser }}
7280
runAsGroup: {{ .Values.lightning.lightning.containerSecurityContext.runAsGroup }}

charts/multiwoven/templates/multiwoven-ingress.yaml

Lines changed: 30 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,24 @@ metadata:
66
labels:
77
{{- include "chart.labels" . | nindent 4 }}
88
annotations:
9-
{{ if .Values.externalDNS.enabled }}
9+
{{- if .Values.externalDNS.enabled }}
10+
{{- $hosts := list
11+
.Values.multiwovenConfig.uiHost
12+
.Values.multiwovenConfig.apiHost
13+
.Values.multiwovenConfig.workerHost
14+
.Values.multiwovenConfig.solidWorkerHost
15+
}}
16+
1017
{{- if .Values.temporal.enabled }}
11-
external-dns.alpha.kubernetes.io/hostname: "{{ .Values.multiwovenConfig.uiHost }},{{ .Values.multiwovenConfig.apiHost }},{{ .Values.multiwovenConfig.workerHost }},{{ .Values.multiwovenConfig.solidWorkerHost }},{{ .Values.multiwovenConfig.temporalUiHost }}"
18+
{{- $hosts = append $hosts .Values.multiwovenConfig.temporalUiHost }}
1219
{{- end }}
13-
{{- if not .Values.temporal.enabled }}
14-
external-dns.alpha.kubernetes.io/hostname: "{{ .Values.multiwovenConfig.uiHost }},{{ .Values.multiwovenConfig.apiHost }},{{ .Values.multiwovenConfig.workerHost }},{{ .Values.multiwovenConfig.solidWorkerHost }}"
20+
21+
{{- if .Values.lightning.enabled }}
22+
{{- $hosts = append $hosts .Values.lightningConfig.lightningHealthHost }}
1523
{{- end }}
16-
{{ end }}
24+
25+
external-dns.alpha.kubernetes.io/hostname: "{{ join "," $hosts }}"
26+
{{- end }}
1727

1828
{{ if .Values.kongIngress.enabled }}
1929
{{- if .Values.kongIngress.plugins.prometheus.enabled }}
@@ -58,6 +68,9 @@ spec:
5868
- {{ .Values.multiwovenConfig.solidWorkerHost }}
5969
{{ if .Values.temporal.enabled }}
6070
- {{ .Values.multiwovenConfig.temporalUiHost }}
71+
{{ end }}
72+
{{ if .Values.lightning.enabled }}
73+
- {{ .Values.lightningConfig.lightningHealthHost }}
6174
{{ end }}
6275
secretName: mw-tls-cert
6376
{{ end }}
@@ -121,3 +134,15 @@ spec:
121134
path: /
122135
pathType: Prefix
123136
{{ end }}
137+
{{ if .Values.lightning.enabled }}
138+
- host: {{ .Values.lightningConfig.lightningHealthHost }}
139+
http:
140+
paths:
141+
- backend:
142+
service:
143+
name: '{{ include "chart.fullname" . }}-lightning'
144+
port:
145+
number: {{ (index .Values.lightning.ports 0).port }}
146+
path: /health
147+
pathType: Prefix
148+
{{ end }}

charts/multiwoven/values.yaml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@ multiwovenConfig:
9797

9898
lightningConfig:
9999
lightningUrl: http://multiwoven-lightning:6000
100+
lightningHealthHost: lightning.multiwoven.com
100101

101102
xpertApiKey: ""
102103
xpertEndpoint: ""
@@ -117,6 +118,11 @@ lightningConfig:
117118
seemoreVlm: ""
118119
seemoreEndpoint: ""
119120

121+
judyApiKey: ""
122+
judyModel: ""
123+
judyEndpoint: ""
124+
125+
120126
automountServiceAccountToken:
121127
enabled: true
122128

0 commit comments

Comments
 (0)