Merge pull request #601 from ryandawsonuk/527-engine-user

option to not set engine user
SeldonIO · May 29, 2019 · c5d7b30 · c5d7b30
2 parents 1b72210 + 38bdb3d
commit c5d7b30
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/helm-charts/seldon-core-operator/templates/controller.yaml b/helm-charts/seldon-core-operator/templates/controller.yaml
@@ -199,8 +199,10 @@ spec:
           value: {{ .Values.engine.image.pullPolicy }}
         - name: ENGINE_CONTAINER_SERVICE_ACCOUNT_NAME
           value: {{ .Values.engine.serviceAccount.name }}
+        {{- if .Values.engine.securityContext.enabled }}
         - name: ENGINE_CONTAINER_USER
           value: '{{ .Values.engine.user }}'
+        {{- end }}
         - name: PREDICTIVE_UNIT_SERVICE_PORT
           value: '{{ .Values.predictiveUnit.port }}'
         - name: ENGINE_SERVER_GRPC_PORT

diff --git a/helm-charts/seldon-core-operator/values.yaml b/helm-charts/seldon-core-operator/values.yaml
@@ -16,6 +16,8 @@ engine:
     path: prometheus
   serviceAccount:
     name: default
+  securityContext:
+    enabled: true
   user: 8888
 image:
   pullPolicy: IfNotPresent