Skip to content

Commit

Permalink
Use the webapp-url when deploying as Enterprise. This is probably bro…
Browse files Browse the repository at this point in the history
…ken for OSS, so this needs some review and suggestions.
  • Loading branch information
bgroff committed Mar 18, 2024
1 parent f9a2b46 commit 8f8b4e5
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
5 changes: 5 additions & 0 deletions charts/airbyte-server/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,11 @@ spec:
value: "true"
{{- end }}
{{- if eq .Values.global.deploymentMode "oss" }}
- name: AIRBYTE_API_HOST
valueFrom:
configMapKeyRef:
name: {{ .Release.Name }}-airbyte-env
key: AIRBYTE_API_HOST
- name: AIRBYTE_VERSION
valueFrom:
configMapKeyRef:
Expand Down
2 changes: 1 addition & 1 deletion charts/airbyte/templates/env-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ data:
KEYCLOAK_INTERNAL_HOST: localhost # just a placeholder so that nginx template is valid - shouldn't be used when edition isn't "pro"
{{- end }}
CONNECTOR_BUILDER_API_HOST: {{ .Release.Name }}-airbyte-connector-builder-server-svc:{{ index .Values "connector-builder-server" "service" "port" }}
AIRBYTE_API_HOST: {{ .Release.Name }}-airbyte-api-server-svc:{{ index .Values "airbyte-api-server" "service" "port" }}
AIRBYTE_API_HOST: {{ printf "%s/api/public/v1" (index $airbyteYmlDict "webapp-url") | quote }}
{{- if $.Values.global.jobs.kube.annotations }}
JOB_KUBE_ANNOTATIONS: {{ $.Values.global.jobs.kube.annotations | include "airbyte.flattenMap" | quote }}
{{- end }}
Expand Down

0 comments on commit 8f8b4e5

Please sign in to comment.