Skip to content

Commit f1e8381

Browse files
authored
[S3] Automount s3 credential secret rather than inline keys (#262)
1 parent 6a47d15 commit f1e8381

File tree

5 files changed

+5
-5
lines changed

5 files changed

+5
-5
lines changed

.github/workflows/ci.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
python-version: "3.x"
2626
check-latest: true
2727
- name: Set up chart-testing
28-
uses: helm/chart-testing-action@e6669bcd63d7cb57cb4380c33043eebe5d111992 #v2.6.1
28+
uses: helm/chart-testing-action@6ec842c01de15ebb84c8627d2744a0c2f2755c9f #v2.8.0
2929
with:
3030
version: v3.10.1
3131
- name: Run chart-testing (list-changed)

charts/mlrun-ce/Chart.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
apiVersion: v1
22
name: mlrun-ce
3-
version: 0.11.0-rc9
3+
version: 0.11.0-rc.10
44
description: MLRun Open Source Stack
55
home: https://iguazio.com
66
icon: https://www.iguazio.com/wp-content/uploads/2019/10/Iguazio-Logo.png

charts/mlrun-ce/templates/_helpers.tpl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -191,7 +191,7 @@ Uses SeaweedFS as the storage backend
191191
{{- if hasKey .Values.mlrun "storageAutoMountParams" -}}
192192
{{ .Values.mlrun.storageAutoMountParams }}
193193
{{- else if not .Values.global.infrastructure.aws.s3NonAnonymous -}}
194-
"aws_access_key={{ include "mlrun-ce.s3.accessKey" . }},aws_secret_key={{ include "mlrun-ce.s3.secretKey" . }},endpoint_url={{ include "mlrun-ce.s3.service.url" . }}"
194+
"secret_name=s3-credentials,endpoint_url={{ include "mlrun-ce.s3.service.url" . }}"
195195
{{- else -}}
196196
"non_anonymous=True"
197197
{{- end -}}

charts/mlrun-ce/templates/config/jupyter-env-configmap.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ metadata:
66
name: jupyter-common-env
77
data:
88
# S3 credentials (AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY, AWS_ENDPOINT_URL_S3)
9-
# are now loaded from the 's3-credentials' Secret via envFrom secretRef
9+
# are loaded from the 's3-credentials' Secret
1010
MLRUN_STORAGE__AUTO_MOUNT_TYPE: {{ default "" .Values.mlrun.storageAutoMountType }}
1111
S3_NON_ANONYMOUS: {{ .Values.global.infrastructure.aws.s3NonAnonymous | toString | title | quote | default "\"True\"" }}
1212
MLRUN_CE__MODE: {{ .Values.jupyterNotebook.ce.mode | default "full" }}

charts/mlrun-ce/templates/config/mlrun-env-configmap.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ data:
1212
MLRUN_HTTPDB__PROJECTS__LEADER: mlrun
1313
MLRUN_HTTPDB__PROJECTS__FOLLOWERS: nuclio
1414
# S3 credentials (AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY, AWS_ENDPOINT_URL_S3)
15-
# are now loaded from the 's3-credentials' Secret via envFrom secretRef
15+
# are loaded from the 's3-credentials' Secret
1616
MLRUN_FUNCTION__SPEC__SERVICE_ACCOUNT__DEFAULT: {{ .Values.mlrun.api.functionSpecServiceAccountDefault | default "" | quote }}
1717
MLRUN_HTTPDB__REAL_PATH: s3://
1818
MLRUN_ARTIFACT_PATH: s3://{{ $bucket_name }}/projects/{{ `{{run.project}}` }}/artifacts

0 commit comments

Comments
 (0)