Skip to content

Commit

Permalink
chore(helm): add configuration to override model-backend host (#462)
Browse files Browse the repository at this point in the history
Because

- In the multi-region architecture, the Instill Model connector needs to
connect to the model-backend through an external network instead of an
internal network, we need to have a configuration for this.

This commit:

- Adds configuration to override the model-backend host.
  • Loading branch information
donch1989 authored Mar 31, 2024
1 parent d4be1eb commit b42e72f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 2 additions & 2 deletions charts/core/templates/pipeline-backend/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,8 @@ data:
key: /etc/instill-ai/vdp/ssl/mgmt/tls.key
{{- end }}
modelbackend:
host: {{ template "core.modelBackend" . }}
publicport: {{ template "core.modelBackend.publicPort" . }}
host: {{ default (include "core.modelBackend" . ) .Values.pipelineBackend.modelBackendHostOverride }}
publicport: {{ default (include "core.modelBackend.publicPort" . ) .Values.pipelineBackend.modelBackendPortOverride }}
{{- if .Values.internalTLS.enabled }}
https:
cert: /etc/instill-ai/vdp/ssl/model/tls.crt
Expand Down
2 changes: 2 additions & 0 deletions charts/core/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -472,6 +472,8 @@ pipelineBackend:
minAvailable:
maxUnavailable:
copyHostDocker: false
modelBackendHostOverride:
modelBackendPortOverride:
# -- The configuration of model-backend
modelBackend:
# -- The image of model-backend
Expand Down

0 comments on commit b42e72f

Please sign in to comment.