Skip to content

Commit

Permalink
[FEAT][HELM] set ES SSL verification (#5807)
Browse files Browse the repository at this point in the history
# Description
Added a variable to set `ARGILLA_ELASTICSEARCH_SSL_VERIFY` in the helm
chart.

_BTW: I have found there's no documentation for the helm values, I'd
love to create a one anytime._
  • Loading branch information
omarmoo5 authored Feb 17, 2025
1 parent a9c80ec commit ce50eb3
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,8 @@ spec:
- name: ARGILLA_ELASTICSEARCH
# TODO: modify this to use externalElasticsearch.host if elasticsearch.useOperator is false
value: {{if .Values.elasticsearch.useOperator}}"http://{{ include "argilla.fullname" . }}-es-http:9200"{{else}}"{{ .Values.externalElasticsearch.host }}:{{ .Values.externalElasticsearch.port }}"{{end}}
- name: ARGILLA_ELASTICSEARCH_SSL_VERIFY
value: {{ .Values.elasticsearch.sslVerify | quote }}
- name: ARGILLA_AUTH_SECRET_KEY
value: {{ .Values.argilla.authSecretKey | quote }}
- name: ARGILLA_REDIS_URL
Expand Down
1 change: 1 addition & 0 deletions examples/deployments/k8s/argilla-chart/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ argilla:
# To disable it, set `elasticsearch.useOperator` to `false`
# and set `externalElasticsearch.host` to the host of your existing Elasticsearch instance.
elasticsearch:
sslVerify: true
useOperator: true
version: 8.5.3
nodeCount: 1
Expand Down

0 comments on commit ce50eb3

Please sign in to comment.