Skip to content

Commit

Permalink
Move Jupyter extraEnv to CM to support bucket_name param
Browse files Browse the repository at this point in the history
  • Loading branch information
alexpirogovski committed Aug 29, 2024
1 parent 7852e60 commit f75ad64
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 12 deletions.
2 changes: 1 addition & 1 deletion charts/mlrun-ce/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
apiVersion: v1
name: mlrun-ce
version: 0.6.4-rc8
version: 0.6.4-rc9
description: MLRun Open Source Stack
home: https://iguazio.com
icon: https://www.iguazio.com/wp-content/uploads/2019/10/Iguazio-Logo.png
Expand Down
5 changes: 4 additions & 1 deletion charts/mlrun-ce/templates/config/jupyter-env-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,12 @@ data:
MLRUN_STORAGE__AUTO_MOUNT_TYPE: s3
S3_NON_ANONYMOUS: {{ .Values.global.infrastructure.aws.s3NonAnonymous | default "True" | quote }}
MLRUN_FUNCTION__SPEC__SERVICE_ACCOUNT__DEFAULT: {{ .Values.mlrun.api.functionSpecServiceAccountDefault }}
MLRUN_FEATURE_STORE__DATA_PREFIXES__DEFAULT: s3:///{{ $bucket_name }/projects/{project}/FeatureStore/{name}/{kind}
AWS_SECRET_ACCESS_KEY: {{ .Values.minio.rootPassword }}
AWS_ACCESS_KEY_ID: {{ .Values.minio.rootUser }}
MLRUN_STORAGE__AUTO_MOUNT_PARAMS: {{ .Values.mlrun.storageAutoMountParams | default "non_anonymous=True" }}
MLRUN_ARTIFACT_PATH: s3://{{ $bucket_name }}/projects/{{ `{{run.project}}` }}/artifacts
MLRUN_ARTIFACT_PATH: s3://{{ $bucket_name }}/projects/{{ `{{run.project}}` }}/artifacts{{run.uid}}
MLRUN_FEATURE_STORE__DATA_PREFIXES__NOSQL: ""
MLRUN_FEATURE_STORE__DEFAULT_TARGETS: parquet
MLRUN_HTTPDB__REAL_PATH: s3://
{{- end}}
10 changes: 0 additions & 10 deletions charts/mlrun-ce/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -190,16 +190,6 @@ jupyterNotebook:
# use this to override mlrunUIURL, by default it will be auto-resolved to externalHostAddress and
# mlrun UI's node port
mlrunUIURL:
extraEnv:
- name: MLRUN_ARTIFACT_PATH
value: s3://mlrun/projects/{{run.project}}/artifacts/{{run.uid}}
- name: MLRUN_FEATURE_STORE__DATA_PREFIXES__DEFAULT
value: s3://mlrun/projects/{project}/FeatureStore/{name}/{kind}
- name: MLRUN_FEATURE_STORE__DATA_PREFIXES__NOSQL
value: ""
- name: MLRUN_FEATURE_STORE__DEFAULT_TARGETS
value: "parquet"

extraEnvKeyValue: {}
envFrom:
- configMapRef:
Expand Down

0 comments on commit f75ad64

Please sign in to comment.