Skip to content

Commit

Permalink
chore(charts): remove host/port override for model and artifact backe…
Browse files Browse the repository at this point in the history
…nd (#1075)

Because

- The host/port override for the model and artifact backends is no
longer needed.

This commit

- Removes the host/port override for the model and artifact backends.
  • Loading branch information
donch1989 authored Aug 25, 2024
1 parent b4a6a8b commit 9ec2142
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 8 deletions.
8 changes: 4 additions & 4 deletions charts/core/templates/pipeline-backend/configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,16 +38,16 @@ data:
key: /etc/instill-ai/vdp/ssl/mgmt/tls.key
{{- end }}
artifactbackend:
host: {{ default (include "core.artifactBackend" . ) .Values.pipelineBackend.artifactBackendHostOverride }}
publicport: {{ default (include "core.artifactBackend.publicPort" . ) .Values.pipelineBackend.artifactBackendPortOverride }}
host: {{ template "core.artifactBackend" . }}
publicport: {{ template "core.artifactBackend.publicPort" . }}
{{- if .Values.internalTLS.enabled }}
https:
cert: /etc/instill-ai/vdp/ssl/artifact/tls.crt
key: /etc/instill-ai/vdp/ssl/artifact/tls.key
{{- end }}
modelbackend:
host: {{ default (include "core.modelBackend" . ) .Values.pipelineBackend.modelBackendHostOverride }}
publicport: {{ default (include "core.modelBackend.publicPort" . ) .Values.pipelineBackend.modelBackendPortOverride }}
host: {{ template "core.modelBackend" . }}
publicport: {{ template "core.modelBackend.publicPort" . }}
{{- if .Values.internalTLS.enabled }}
https:
cert: /etc/instill-ai/vdp/ssl/model/tls.crt
Expand Down
4 changes: 0 additions & 4 deletions charts/core/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -453,10 +453,6 @@ pipelineBackend:
spec:
minAvailable:
maxUnavailable:
modelBackendHostOverride:
modelBackendPortOverride:
artifactBackendHostOverride:
artifactBackendPortOverride:
minio:
port: 9000
rootuser: minioadmin
Expand Down

0 comments on commit 9ec2142

Please sign in to comment.